Junio C Hamano <gitster@xxxxxxxxx> writes: > Johannes Schindelin <Johannes.Schindelin@xxxxxx> 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). OK! I can also say that it looks better in my eyes to do it like this, so, Junio, please use this version of the patch, and forget about v2 of the patch posted some time later. -- kjetil -- 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