Re: [PATCH v2 2/3] server: Add special case destroy signal emitter

2018-04-16 Thread Simon Ser
> In the past much code (weston, efl/enlightenment, mutter) has > freed structures containing wl_listeners from destroy handlers > without first removing the listener from the signal. As the > destroy notifier only fires once, this has largely gone > unnoticed until recently. > > Other code does

[PATCH v2 2/3] server: Add special case destroy signal emitter

2018-04-16 Thread Derek Foreman
In the past much code (weston, efl/enlightenment, mutter) has freed structures containing wl_listeners from destroy handlers without first removing the listener from the signal. As the destroy notifier only fires once, this has largely gone unnoticed until recently. Other code does not (Qt, wlroo