On Wednesday, November 7, 2007 at 11:10:45 (+0000) Johannes Schindelin writes:
>Hi,
>
>you still have quite a number of instances where you wrap just one line
>into curly brackets:
>
> if (bla) {
> [just one line]
> }
I've always found this a thoughtful practice. It helps ensure nobody writes:
if (bla)
just_one_line();
/* perhaps a comment, other stuff ... */
just_another_line();
which I've seen happen countless times. It also is nice for others who
come along and extend the branch from just one line to multiple ones,
as the brackets are already in place.
Why do you find it objectionable?
Bill
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
