From: Jeff King Date: Sat, 12 Jun 2010 04:11:46 GMT Subject: Re: [PATCH v2 2/3] textconv: support for blame Message-ID: <20100612041146.GB9419@coredump.intra.peff.net> In-Reply-To: <7viq5pgma4.fsf@alter.siamese.dyndns.org> On Fri, Jun 11, 2010 at 04:52:19PM -0700, Junio C Hamano wrote: > Axel Bonnet writes: > > > + switch (userdiff_config(var, value)) { > > + case 0: break; > > + case -1: return -1; > > + default: return 0; > > + } > > Style: > > switch (userdiff_config(var, value)) { > case 0: > break; > case -1: > return -1; > default: > return 0; > } This is cut-and-paste from some of my code in git_diff_basic_config. I dunno if it is worth style-fixing that one, too. -Peff