[PATCH v2] Re: avoid useless if-before-free tests
Jim Meyering
jim@meyering.net
Tue Mar 8 19:13:00 GMT 2011
Rainer Orth wrote:
> Jim Meyering <jim@meyering.net> writes:
>>> the change at hand. And please don't change the alignment of entries
>>> with multiple email addresses.
>>
>> Changing 8-spaces to a TAB does not affect alignment when you're
>> looking at the ChangeLog file itself with standard tab setting.
>>
>> Perhaps you looked at a hunk like the following and mistook it
>> for one that introduces an alignment change?
>>
>> 2011-11-04 Eric Botcazou <ebotcazou@adacore.com>
>> - Jakub Jelinek <jakub@redhat.com>
>> + Jakub Jelinek <jakub@redhat.com>
>>
>> It does not.
>
> I'm pretty sure it does: before, you have 12 SPC, afterwards you have
> TAB + 3 SPC, which is equivalent to 11 SPC in my book.
Here's the precise excerpt from my patch:
2011-11-04 Eric Botcazou <ebotcazou@adacore.com>
- Jakub Jelinek <jakub@redhat.com>
+ Jakub Jelinek <jakub@redhat.com>
That has TAB + 4, so induced no alignment change.
> I honestly don't see the point of this whitespace change unless done
> across all ChangeLogs, not just a few that you happen to touch.
As I said, it's gone, now, from my patch.
If no one objects, I'll normalize all ChangeLog files.
More information about the Gcc-patches
mailing list