Home » Mailing list archives » git » 2007 » September » 16

Re: [PATCH 2/3] git-apply: complain about >=8 consecutive spaces in initial indent

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Martin Langhoff <martin.langhoff@...>
To: J. Bruce Fields <bfields@...>
Cc: Junio C Hamano <junkio@...>, <git@...>
Subject: Re: [PATCH 2/3] git-apply: complain about &gt;=8 consecutive spaces in initial indent
Date: Sunday, September 16, 2007 - 7:24 pm

On 9/17/07, J. Bruce Fields <bfields@citi.umich.edu> wrote:

I do quite a bit of hacking on "spaces-for-indentation" projects and
still use stripspace to cleanup my patches. So no, thanks.

Perhaps split it off to a separate option? I'm not opposed to the
functionality per-se, but don't put together with
trailing-space-trimming. It's a different beast. Everyone agrees
trimming trailing spaces as much as everyone disagrees on
tabs-vs-spaces.

cheers,



m
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
whitespace-stripping, J. Bruce Fields, (Sun Sep 16, 6:48 pm)
[PATCH 1/3] git-apply: fix whitespace stripping, J. Bruce Fields, (Sun Sep 16, 6:49 pm)
[PATCH 2/3] git-apply: complain about &gt;=8 consecutive spa..., J. Bruce Fields, (Sun Sep 16, 6:49 pm)
[PATCH] git-diff: complain about &gt;=8 consecutive spaces i..., Junio C Hamano, (Tue Oct 2, 9:00 pm)
Re: [PATCH 2/3] git-apply: complain about &gt;=8 consecutive..., Junio C Hamano, (Sun Sep 16, 8:24 pm)
Re: [PATCH 2/3] git-apply: complain about &gt;=8 consecutive..., J. Bruce Fields, (Sun Sep 16, 10:44 pm)
Re: [PATCH 2/3] git-apply: complain about >=8 consecutive..., Martin Langhoff, (Sun Sep 16, 7:24 pm)
Re: [PATCH 2/3] git-apply: complain about &gt;=8 consecutive..., J. Bruce Fields, (Sun Sep 16, 10:45 pm)
[PATCH 3/3] git-apply: add tests for stripping of leading an..., J. Bruce Fields, (Sun Sep 16, 6:49 pm)
Re: [PATCH 3/3] git-apply: add tests for stripping of leadin..., Krzysztof Halasa, (Mon Sep 17, 10:16 am)
Re: [PATCH 3/3] git-apply: add tests for stripping of leadin..., J. Bruce Fields, (Mon Sep 17, 11:02 am)
Re: [PATCH 3/3] git-apply: add tests for stripping of leadin..., Krzysztof Halasa, (Mon Sep 17, 7:44 pm)
Re: [PATCH 3/3] git-apply: add tests for stripping of leadin..., J. Bruce Fields, (Mon Sep 17, 8:53 pm)