Mike Frysinger wrote:
On Saturday 31 October 2009 22:03:45 Tom wrote:
Mike Frysinger wrote:
On Saturday 31 October 2009 13:37:45 Tom Rix wrote:
- } else {
usbtty_putc(c);dont need those braces (same goes for a few other hunks here) -mike
This is done because if-statement above use braces. If this was a simple if (foo) smt_1 else smt_2
I would have not use braces.
the trailing "else" clause is one statement. the relationship to the previous if section doesnt matter. common style convention is: if (foo) { statement_1; statement_2; } else statement_3; -mike
Please see
http://www.kernel.org/doc/Documentation/CodingStyle
Do not unnecessarily use braces where a single statement will do.
if (condition) action();
This does not apply if one branch of a conditional statement is a single statement. Use braces in both branches.
if (condition) { do_this(); do_that(); } else { otherwise(); }
Tom