<div dir="ltr"><div><br>I also spent some time trying to assess all possible cases (with very limited knowledge on xlib internals), and I'm pretty sure Jonas has dedicated even more time than the both of us did, but that's not the issue here.<br><br></div>I agree the patch is hard, and maybe a more satisfactory solution could be imagined by redesigning the 32-bit sequence numbers so that it aligns better with xcb, but at some point a decision needs to be taken. In that sense it *is* probably better to push the patch than to leave the current situation as is where the application program just simply crashes...<br><br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On 27 September 2014 05:23, Keith Packard <span dir="ltr"><<a href="mailto:keithp@keithp.com" target="_blank">keithp@keithp.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>> writes:<br>
<br>
> On Fri, Sep 26, 2014 at 3:40 AM, Jan Smout <<a href="mailto:smout.jan@gmail.com">smout.jan@gmail.com</a>> wrote:<br>
>> Keith Packard doesn't seem very responsive (as in 'completely ignoring the<br>
>> subject')<br>
><br>
> Perhaps you should try Ccing him? (now Cc'd)<br>
<br>
</div></div>The problem is that reviewing this patch is *really hard*. The last<br>
time, I think I spent a solid couple of days thinking about this and<br>
making sure I'd caught all of the cases. I'm still not sure it's right,<br>
but I guess it's probably better than what we have?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
<a href="mailto:keith.packard@intel.com">keith.packard@intel.com</a><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>Life is complex, it has a real part and an imaginary part.
</div>