Hi,
On Wed, 7 Nov 2007, Bill Lear wrote:
> On Wednesday, November 7, 2007 at 11:10:45 (+0000) Johannes Schindelin writes:
>
> > 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();
But if there is only one line and you fail to add curly brackets when
adding a second line, well, uhm, then I cannot help you with anything.
BTW I was talking about _one_ line, not a line and another one with a
comment.
> 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.
The fact is: these lines will stay single lines most likely for eternity.
> Why do you find it objectionable?
It distracts. It's ugly. It's unnecessary.
Ciao,
Dscho
-
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
