Code review comment for lp:~vanvugt/compiz-core/fix-880707.2

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

"I forgot to mention - the check would have to be moved to before priv->reschedule has been set to false (I've roughly transposed the code from my branch to yours)"

That still makes no sense in my mind. But if you can reproduce the slow animations bug consistently, and have tested and verified your fix works then go nuts. If however this is another code change you're proposing without actually testing it, then please reconsider.

« Back to merge proposal