At 1252069260 time_t, Uli Schlachter wrote: > IMHO this is the proper fix for the "c.screen == <some huge number>" bug I had > recently when I accidentally used the client in client::new.
Unfortunately, it's not the proper fix. If you call add_signal() on manage, will miss some property:: signals because some are emitted before "manage" is emitted. This is what we used new rather than manage in some awful code. The client code should be bulletproof about doing stuff on a client that is *being set up to be managed*. I've fix the issue you found, there's *maybe* more. And if there's more, we should fix them. :) -- Julien Danjou // ᐰ <[email protected]> http://julien.danjou.info // 9A0D 5FD9 EB42 22F6 8974 C95C A462 B51E C2FE E5CD // When I get sad, I stop being sad and be awesome instead. True story.
signature.asc
Description: Digital signature
