[BusyBox] Re: ifconfig: Clean up. [PATCH]
Shaun Jackman
sjackman at gmail.com
Mon Jul 25 17:18:50 UTC 2005
On 7/25/05, Tito <farmatito at tiscali.it> wrote:
> Please use brackets on all if and else statements, even if it is only one
> line. Example:
...
If either foo or stmt1 are simple, I much prefer...
if (foo) stmt1;
If both foo and stmt1 are complex, I prefer...
if (foo)
stmt1;
I almost never prefer...
if (foo) {
stmt1;
}
I don't find the extra punctuation adds to the readability of the
code. I don't like extra parenthesis around the argument to the
keywords return and sizeof for the same reason.
return n;
not
return (n);
and
sizeof buf
not
sizeof (buf)
Cheers,
Shaun
More information about the busybox
mailing list