Merge from Artem's fork

[jbkempf]

This needs reviewing :)

[jbkempf]

Well, in fact, most of it is not good. But there are a couple of things to cherry-pick.

[jbkempf]

@elthariel no, most of the branch is very bad. There is an interesting patch about move(), but the rest is almost all useless...

[elthariel]

There's also the asn1 thread safety part which is good

[elthariel]

and the logoff

[jbkempf]

@elthariel then we should cherry-pick those :)

[elthariel]

Yeah. I started to have a look by but that won't be easy

On Mon, May 8, 2017, 17:41 Jean-Baptiste Kempf [email protected] wrote:

@elthariel https://github.com/elthariel then we should cherry-pick those :)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/videolabs/libdsm/pull/105#issuecomment-299904782, or mute the thread https://github.com/notifications/unsubscribe-auth/AAJa866DdOKNBtDl4jlTmKawE8xbPmEfks5r3zfEgaJpZM4MVSR0 .

[jbkempf]

@Kxuan do you now what we must cherry-pick?

[Kxuan]

@jbkempf acturally, I dont known.. There are too many commits that I do not need. I just merge the 40c2404 to my fork...

[jbkempf]

@Kxuan what is there in your fork? Just 40c2404 ?

[Kxuan]

Yes. Only 4e226691e7359d798415dc09c7c901d9370ed666 is picked

[chouquette]

4e22669 has been merged