<div dir="ltr"><div>O and btw, if there is no better way to query the pointer, its still acceptabel and I'll merge the patch in master ;-).<br><br></div>Nick<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Sat, Nov 16, 2013 at 11:49 AM, Nick Schermer <span dir="ltr"><<a href="mailto:nick@xfce.org" target="_blank">nick@xfce.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div>Nice patch.<br><br>Small note about the code, please avoid "//" comments.<br><br>Another note is the timeout, I understand why you do this but the 32ms feels a bit random (1000/60*2 is a better way to indicate you run it half the fps for example). That said XQueryPointer is not the fastest function on this planet so cpu usage becomes "high" why a user starts to zoom why not moving the mouse. Maybe it is possible to use the mousemotion event for that, but this is probably limited because you need to grab the pointer for the full list of events. Duno if there are other options for this?<br>

<br></div><div>The change to the clipping seems correct since paint_win only works on the rootBuffer. That said the function also applies its own clipping before rendering so it might be redundant as well.<span class="HOEnZb"><font color="#888888"><br>
</font></span></div><span class="HOEnZb"><font color="#888888"><div><br>
</div>Nick<br></font></span></div>
</blockquote></div><br></div>