From: Junio C Hamano <gitster@pobox.com>
Date: Sun, 07 Jun 2009 16:30:41 GMT
Subject: Re: [PATCH] symlinks.c: small style cleanup
Message-ID: <7veitwatvy.fsf@alter.siamese.dyndns.org>
In-Reply-To: <alpine.DEB.1.00.0906071616060.26154@pacific.mpi-cbg.de>

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

> On Sun, 7 Jun 2009, Kjetil Barvik wrote:
>
>> Add {}-braces around an else-part, where the if-part already has
>> {}-braces.
>
> This is the wrong way round.  We prefer _not_ to add unnecessary braces, 
> but do tolerate them from time to time.

I am at fault not spelling this out so far, but we prefer to match the
kernel style of having {} around a single-statment "else" body when the
corresponding "if" side needs one (or vice versa).