Code review comment for lp://staging/~hikiko/compiz/0.9.11-fix-1245886

Revision history for this message
MC Return (mc-return) wrote :

+1.

Are the changes (additional brackets) in src/window.cpp really needed ?

So adding

if (!activeWindow)
    return;

to decor.cpp, DecorScreen::handleEvent () would be enough imho...

The commit message could be nicer also ;)

review: Approve

« Back to merge proposal