Code review comment for lp:~vanvugt/compiz-core/fix-timer-infinite-loop

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

Hey Daniel,

Thank you for the fix to what looks like it was quite a complicated problem! I have been trying to debug some of the inaccuracies with the timers for some time now.

Could you attach a simple plugin to this which shows the old behaviour happening? OR even better, would be to add a unit test in tests/timer/

Cheers,

Sam

review: Needs Information

« Back to merge proposal