This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 2/4] Docs: extend.texi: Remove trailing blanks from lines
- From: Mike Stump <mikestump at comcast dot net>
- To: Michael Witten <mfwitten at gmail dot com>
- Cc: Gerald Pfeifer <gerald at pfeifer dot com>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 4 May 2011 10:44:14 -0700
- Subject: Re: [PATCH 2/4] Docs: extend.texi: Remove trailing blanks from lines
- References: <378e16b9-5d40-427c-8b4e-00700b2ad30c-mfwitten@gmail.com> <eeb6242f-997b-44f4-8f8f-90a559362ea1-mfwitten@gmail.com> <alpine.LNX.2.00.1105031943140.2899@gerinyyl.fvgr> <bf39f368-182b-4749-98b9-550c627df8a9-mfwitten@gmail.com>
On May 3, 2011, at 10:27 PM, Michael Witten wrote:
> On Tue, 3 May 2011 22:52:25 +0200 (CEST), Gerald Pfeifer wrote:
>
>> <On Wed, 27 Apr 2011, Michael Witten wrote:
>>> sed -i "s/[ $(printf '\t')]\{1,\}\$//" trunk/gcc/doc/extend.texi
>>
>> I believe we usually don't do whitespace changes just for the sake
>> of cleaning up things.
>>
>> Unless someone else indicates otherwise, I believe this patch is
>> rejected.
>
> To what do we owe this tradition other than laziness?
By flat out rejecting style fixing patches, you preserve the annotations made by vc-blame (svn blame). That's the only reason that I am aware of. Laziness can't be the reason to reject the hard work of someone that wants to clean up the code.
Like others, I don't find that compelling enough personally.