<div dir="ltr">Oh, good catch; just reviewed it. Thanks!</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 28, 2016 at 10:20 AM, Daniel Stone <span dir="ltr"><<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Dima,<br>
<span class=""><br>
On 24 November 2016 at 13:13, Dima Ryazanov <<a href="mailto:dima@gmail.com">dima@gmail.com</a>> wrote:<br>
> When a window is being closed, the frame_done callback often runs after<br>
> the output is already destroyed, i.e:<br>
><br>
>   wayland_output_start_repaint_<wbr>loop<br>
>   input_handle_button<br>
>     wayland_output_destroy<br>
>   frame_done<br>
><br>
> To fix this, destroy the callback before destroying the output.<br>
><br>
> (Also, fix the type of output in frame_done: it's passed in<br>
> a wayland_output, not a weston_output.)<br>
<br>
</span>I accidentally pushed this, when I meant to push the gl-renderer patch<br>
instead. Turns out this breaks Pixman; I've sent a follow-up patch<br>
which fixes this. Can you please review it when you can?<br>
<br>
Cheers,<br>
Daniel<br>
</blockquote></div><br></div>