Code review comment for lp:~oif-team/grail/trunk.fixes

Revision history for this message
Chase Douglas (chasedouglas) wrote :

In line 371 of the patch below, there's an ambiguity in the if statement "if (a || b && c)". This should be cleaned up.

I need to test this branch, but I think the changes look reasonable. However, I can't approve the request until it also handles the global flag. This is something I think we will really need in natty, since we're going to have more applications using gestures.

review: Needs Fixing

« Back to merge proposal