Re: [PATCH] remove trailing whitespace

From: Russell King (rmk@arm.linux.org.uk)
Date: Sun Nov 25 2001 - 07:25:57 EST


On Sun, Nov 25, 2001 at 02:00:43PM -0200, vda wrote:
> Isn't it easier to write a script 'clean_trailing_ws' (or whatever)
> and add it to scripts/ ?
> Linus can apply such 'cleaning' scripts at times like 2.4 -> 2.5

It's best to have a script that you pass all patches through which
strips off trailing space on any line that gets added (you can't
do it blindly since the context lines must obviously match the
original, as must the lines being removed).

I do this automatically when people send stuff to my patch system.
Appears to work well.

It solves the problem of the trailing space getting into the source
in the first place, and doesn't produce a huge 25MB patch.

I suppose you could also remove anything which matched 1-7 spaces
and a tab character, but this might be less reliable. Obviously
you can't replace 8 spaces with a tab character, since it might
be part of a printk string.

If anyone's interested, this is a fragment from the perl script which
processes incoming patches. Might be useful to someone.

        #
        # And now the actual patch itself
        #
        while (<STDIN>)
        {
                chomp;
                s/\s+$// if m/^\+/;
                $mail .= "$_\n";
                $patchtext .= "$_\n";
        }

I'm sure someone can come up with something more efficient. 8)

--
Russell King (rmk@arm.linux.org.uk)                The developer of ARM Linux
             http://www.arm.linux.org.uk/personal/aboutme.html

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Fri Nov 30 2001 - 21:00:18 EST