Clear settings['updates']['callback'] in __wakeup

[andris-zalitis]

I pass my object and function name to updates callback. Settings of MadelineProto are serialized. So when deserializing MadelineProto, it restores reference to object from previous session. Shouldn't we clear it?

I am having particular issue with this because I have pthreads module installed on my server (though I don't use pthreads in MadelineProto) and because of that, I sometimes get this error:

Uncaught RuntimeException: pthreads detected an attempt to connect to an object which has already been destroyed in ...

[danog]

That error is not caused by the callback, but by (one or more) bugs in pthreads, the same bugs that prevent MadelineProto from being threaded: I can't find a way to reproduce them with separate code snippets, so I can't open a proper issue @ krakjoe/pthreads, and make them fix everything. Some help to help reproduce these bugs would sure be appreciated; for now, refer to #4

[andris-zalitis]

@danog

OK, disregarding pthreads, the question remains - shouldn't we clear the reference to updates callback object after deserialization?

[danog]

Possibly, even if I can't find a reason for doing that.

[ghost]

@danog Setting a new callback function(?)

[danog]

Serialization already throws an exception if we try to serialize a function callback, so clearing it on __wakeup is pointless.
Use static class methods as update callbacks.