Re: [PATCH wayland v2] event-loop: Dispatch idle callbacks twice

2015-01-28 Thread Bryce Harrington
On Wed, Jan 28, 2015 at 06:16:16PM +0200, Giulio Camuffo wrote: > Forgot the > > Reviewed-by: Giulio Camuffo Thanks, applied. 7575e2e..5ec8062 master -> master I think this is one of those things that'll be easier to prove right in practice rather than on paper, so let's put this through b

Re: [PATCH wayland v2] event-loop: Dispatch idle callbacks twice

2015-01-28 Thread Giulio Camuffo
Forgot the Reviewed-by: Giulio Camuffo 2015-01-28 17:58 GMT+02:00 Giulio Camuffo : > 2015-01-28 17:49 GMT+02:00 Derek Foreman : >> On 28/01/15 09:39 AM, Giulio Camuffo wrote: >>> 2015-01-28 17:25 GMT+02:00 Derek Foreman : To fix a shutdown crash in weston's x11 compositor I want to move th

Re: [PATCH wayland v2] event-loop: Dispatch idle callbacks twice

2015-01-28 Thread Giulio Camuffo
2015-01-28 17:49 GMT+02:00 Derek Foreman : > On 28/01/15 09:39 AM, Giulio Camuffo wrote: >> 2015-01-28 17:25 GMT+02:00 Derek Foreman : >>> To fix a shutdown crash in weston's x11 compositor I want to move the >>> weston X window close to an idle handler. >>> >>> Since idle handlers are processed at

Re: [PATCH wayland v2] event-loop: Dispatch idle callbacks twice

2015-01-28 Thread Derek Foreman
On 28/01/15 09:39 AM, Giulio Camuffo wrote: > 2015-01-28 17:25 GMT+02:00 Derek Foreman : >> To fix a shutdown crash in weston's x11 compositor I want to move the >> weston X window close to an idle handler. >> >> Since idle handlers are processed at the start of an event loop, the >> handler that d

Re: [PATCH wayland v2] event-loop: Dispatch idle callbacks twice

2015-01-28 Thread Giulio Camuffo
2015-01-28 17:25 GMT+02:00 Derek Foreman : > To fix a shutdown crash in weston's x11 compositor I want to move the > weston X window close to an idle handler. > > Since idle handlers are processed at the start of an event loop, the > handler that deals with window close will run at the start of the

[PATCH wayland v2] event-loop: Dispatch idle callbacks twice

2015-01-28 Thread Derek Foreman
To fix a shutdown crash in weston's x11 compositor I want to move the weston X window close to an idle handler. Since idle handlers are processed at the start of an event loop, the handler that deals with window close will run at the start of the next input_loop dispatch, after which the dispatche