Xfwm4 compositor zooming patch

Nick Schermer nick at xfce.org
Sat Nov 16 11:49:39 CET 2013


Nice patch.

Small note about the code, please avoid "//" comments.

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?

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.

Nick
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.xfce.org/pipermail/xfce4-dev/attachments/20131116/4c18a5e0/attachment.html>


More information about the Xfce4-dev mailing list