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

Attachment: 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

Reply via email to