On Tue, Jul 17, 2012 at 11:26 AM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote: > On Mon, Jul 16, 2012 at 06:57:45PM +0200, Marc-André Lureau wrote: >> When display is released, detach signal automatically > > Same question as the similar patch before, why? I'm assuming that it's > because we are going to handle displays dynamically, and if we don't do > that, we'll sometimes end up with signals using dead objects? Yeah, I don't have a backtrace, but you can get into various crash related to not cleaning up signal handlers properly. In general, this signal_connect helper is really awesome, as you don't have to worry about either emitter or receiver being dead, but some time you may prefer to actually crash if something goes wrong. So a systematic replace isn't what I would recommend. But for Spice channel & display, it is needed to cleanup if either of the two is destroyed. -- Marc-André Lureau
