Re: the war on trailing whitespace
- From
Linus Torvalds <torvalds@osdl.org>
- Date
- Feb 26, 2006, 20:16 UTC
- Message-ID
- <Pine.LNX.4.64.0602261213340.22647@g5.osdl.org>
- In-Reply-To
- <20060226103604.2d97696c.akpm@osdl.org>
On Sun, 26 Feb 2006, Andrew Morton wrote:
> > Thanks. But it defaults to nowarn. Nobody will turn it on and nothing > improves.
Few enough people run "git-apply" on its own. Most people (certainly me) end up using it through some email-applicator script or other. So the plan was that the --whitespace=warn/error flag would go there, and that git-apply by default would work more like "patch".
But hey, I have no strong preferences, and it's easy enough to make the default be warn (and add a "--whitespace=ok" flag to turn it off).
Personally, I don't mind whitespace that much. In particular, I _suspect_ I often have empty lines like
int i; i = 10;
where the "empty" line actually has the same indentation as the lines around it. Is that wrong? Perhaps.
Linus