Code review comment for lp://qastaging/~smspillaz/compiz-core/fix_891591

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

I don't need to fully understand this code to say:

1. The change to src/window.cpp looks like a mistake so should be removed.

2. Are you sure this proposal fixes bug 891591? Or is that a typo and its meant to fix a different bug?

review: Needs Fixing

« Back to merge proposal