[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: 2273 lines in 170 files consists of only space and tabulators
From: |
Felix Zielcke |
Subject: |
Re: 2273 lines in 170 files consists of only space and tabulators |
Date: |
Wed, 10 Jun 2009 23:04:56 +0200 |
Am Samstag, den 06.06.2009, 21:53 -0400 schrieb Pavel Roskin:
> On Sat, 2009-06-06 at 12:25 +0200, Felix Zielcke wrote:
> > address@hidden:~/grub/grub2.git$ rgrep -E "^[[:blank:]]+$" *|wc -l
> > 2273
> > address@hidden:~/grub/grub2.git$ rgrep -E "^[[:blank:]]+$" *|sed -e
> > 's/:.*//'|uniq|wc -l
> > 170
> >
> > Anyone wants me to do
> > find . -type f -print0 | xargs -0 sed -i -r -e 's/^[[:blank:]]+$//'
> > or does someone object?
> > [:blank:] doestn't contain these ^L chars.
> >
> > By the way `git diff --color' shows if you introduce such a line with
> > red, but strangely not when you remove one.
>
> Since we have linear development, it shouldn't be a problem for merging
> patches. Besides, "patch" can be told to ignore whitespace differences.
>
> However, I would prefer that we remove all trailing whitespace, not just
> that on empty lines. That is, remove "^" from the sed expression.
Ok I just commited this.
> As for the script, make sure that it's not run on any special files used
> by Subversion or git. Maybe use DISTLIST.
Thanks for the hint with DISTLIST. gendistlist.sh needed again an
update.
--
Felix Zielcke