Code review comment for lp:~smspillaz/compiz-core/compiz-core.fix_880707.2.test

Revision history for this message
Sam Spilsbury (smspillaz) wrote :

Sure no problem.

One thing I was going to ask based on something I came across while testing this. The composite plugin has an option to throttle frame rates to some specified rate, eg 1 to 200 frames per second. However, with the current code this does not happen if the paint handler is doing vsync. I think it should be possible to limit the fps to a certain number while still doing vsync, but I'm wondering if this is the correct behaviour. Let me know when you have time to.

« Back to merge proposal