Code review comment for lp:~charlesk/indicator-display/adbd-client-test-failure

Revision history for this message
dobey (dobey) wrote :

Generally looks OK to me, but I noticed many uses of if or if/else without curly braces around a single result statement. I think it'd be better to use braces or convert these statements to the ternary operator.

« Back to merge proposal