Code review comment for lp:~sil2100/compiz-core/cherry_3236_3237

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

The ugly refactoring in src/event.cpp of mine is not required and only adds risk if you have the setName change already.

However, this fix is approved upstream, it's still smaller than many other changes being backported, and I'm running it right now and don't see any problems (other than bug 1014957).

review: Approve

« Back to merge proposal