Merge lp:~alanbell/compiz/fix930783 into lp:compiz/0.9.8

[Alan Bell]

makes compiz enhanced zoom and show mouse plugins considerably smoother to use

[Daniel van Vugt (vanvugt)]

Interesting. Though I believe there are bugs related to mousepoll where it keeps polling when it shouldn't. So this fix would make those much worse. For example: bug 961202

[Alan Bell (alanbell)]

good reason to fix those other bugs then! I haven't noticed any performance problems myself, CPU use doesn't change. I can try some powertop stuff to see what happens when it is idle/locked with 1ms polling time.

[Daniel van Vugt (vanvugt)]

By the way, polling more than once per video frame is pointless. On a 60Hz monitor, that means anything more often than (less than) 16ms. Please try raising the interval up to 10ms so it will be smooth on monitors up to 100Hz. That will still ensure mousepoll gets at least one update per frame on almost all monitors.

And please also let us know which bug(s) in launchpad this fixes. It's not bug 30783 :)

review: Needs Information

[Sam Spilsbury (smspillaz)]

Probably the perceived unsmoothness is because the period of the polling doesn't line up with the period of the redraw. I'm not sure about the best way to fix that, as its the polling for a changed position that causes the redraw. We'll need to have a think about it.

[Daniel van Vugt (vanvugt)]

Please modify the default value to 10ms. And please resubmit from a difference branch name since it's not bug 30783.

If you can find any existing bugs this branch fixes then please link them to the branch.

review: Needs Fixing

[Alan Bell (alanbell)]

typo, it was bug 930783 I will make those changes.

[lp:~alanbell/compiz/fix930783]

3251. By Alan Bell on 2012-06-22

10ms is also smooth, should be one poll per frame on almost all output devices still

[Daniel van Vugt (vanvugt)]

Thanks. Next time please click "Resubmit proposal" so it doesn't stay in the old state of "Needs fixing" (and could be potentially overlooked).

review: Approve

=== modified file 'plugins/mousepoll/mousepoll.xml.in'
--- plugins/mousepoll/mousepoll.xml.in	2012-05-16 17:43:36 +0000
+++ plugins/mousepoll/mousepoll.xml.in	2012-06-22 08:06:21 +0000
@@ -16,7 +16,7 @@
 	    <option type="int" name="mouse_poll_interval">
 		<_short>Mouse Poll Interval</_short>
 		<_long>How often to poll the mouse position, in miliseconds. Reduce this to reduce choppy behavior.</_long>
-		<default>40</default>
+		<default>10</default>
 		<min>1</min>
 		<max>500</max>
 	    </option>