<div dir="ltr">Hi Roman,<div class="gmail_extra"><br><div class="gmail_quote">On 18 April 2018 at 20:34, Roman Gilg <span dir="ltr"><<a href="mailto:subdiff@gmail.com" target="_blank">subdiff@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
</span>I believe this check should be ORed with xwl_window->window == window.<br>
Otherwise in case the top parent window is unrealized first,<br>
xwl_window->present_window != window holds here and afterwards<br>
xwl_present_cleanup won't be called again since the xwl_window is<br>
already gone. I.e. an existing frame callback wouldn't be destroyed in<br>
this case.<br></blockquote><div><br></div><div>Sure, I've sent a new patch to do that, but I reckon this can wait after the release, we might leak a frame callback in some cases (although valgrind did not really complain yet), but we won't crash anymore.</div><div><br></div><div>Cheers,</div><div>Olivier</div></div></div></div>