Re: [JGIT PATCH v2 07/24] Added findWorkTree method to Repository class.

From: Miklos Vajna <vmiklos@frugalware.org>
Date: May 23, 2008, 18:31 UTC
Message-ID: <20080523183151.GT27724@genesis.frugalware.org>
In-Reply-To: <4837090F.4000307@web.de>

On Fri, May 23, 2008 at 08:12:31PM +0200, Florian Köberle <FloriansKarten@web.de> wrote:

Show 24 quoted lines

> Also it is better extenable:
> 
> if (a) {
> 	something0();
> }
> 
> if (a) {
> 	something0():
> +	something1();
> }
> 
> compared too:
> 
> -if (a)
> +if (a) {
> 	something0();
> +	something1();
> + }
> 
> 
> it's even possible that someone does it wrong:
> if (a)
> 	something0():
> 	something1();

Different programmers have different coding style. A standard is needed, otherwise different parts of the code will have different style. It's quite normal if a contributor has to code in the project's coding style, rather than his own one, I think.

Also I don't think "but this style is confusing for newbies" is a valid argument, newbies should learn to read code, rather than others need to write newbie-friendly code. ;-)