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
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot