Re: [PATCH] Make git-clean a builtin

From: Bill Lear <rael@zopyra.com>
Date: Nov 7, 2007, 13:29 UTC
Message-ID: <18225.48553.44088.269677@lisa.zopyra.com>
In-Reply-To: <Pine.LNX.4.64.0711071110040.4362@racer.site>

On Wednesday, November 7, 2007 at 11:10:45 (+0000) Johannes Schindelin writes:

Show 8 quoted lines

>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