Code review comment for lp://qastaging/~muktupavels/compiz/ccsm-key-error

Revision history for this message
Dmitry Shachnev (mitya57) wrote :

Tip for next time: if you add parentheses then you don't need to escape newlines:

if (self.CurrentPlugin is not None
    and self.CurrentPlugin.Name in self.Results
    and self.CurrentGroup in self.Results[self.CurrentPlugin.Name]):
    ...

Like you wrote is also fine though.

review: Approve

« Back to merge proposal