<div dir="ltr">Hi Daniel,<br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 19, 2018 at 5:24 PM, 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"><span class="">[...]<br>
</span>The callback is passed as an argument to the function directly, so you<br>
could just destroy that instead.<br></blockquote></div><br></div><div class="gmail_extra">Sure, but that wouldn't solve the problem which is the callback occurs after the “xwl_window” is freed, meaning that we access freed memory here. In other words, we could do that (and I agree it would be more elegant) but that would probably crash on the next line or so.<br><br></div><div class="gmail_extra">Removing all frame callbacks when un-realizing the window as done with the other patch that was merged (thanks Adam!) is safer.<br><br></div><div class="gmail_extra">Cheers,<br></div><div class="gmail_extra">Olivier<br></div></div>