Re: [PATCH][2.5] quotemarks and trailing whitespaces (1st, revisited)

From: Padraig Brady (padraig@antefacto.com)
Date: Mon Jun 24 2002 - 09:53:03 EST


Pavel Machek wrote:
> Hi!
>
>
>>I redid the quotemark patch. Since I'm a lazy typist, I had a script which
>>removed all whitespaces before virtual or real newline characters. Does
>>this one look OK to you?
>
>
> Perhaps such patch should go to scripts/ in distribution, so when
> someone finishes big cleanup for driver can run it at the same time?
> Pavel

This thread is probably of interest.
http://marc.theaimsgroup.com/?l=linux-kernel&m=100653615123970&w=2
It was just when 2.5.0 came out since I thought it was the
most appropriate time for something like this.
Anyway the simple script I used is attached.
Note for 2.5.0 it removed 224,654 bytes.

Padraig.


#!/bin/sh
# Remove trailing whitespace. By default it runs
# in the current directry, on all files, but you
# can change this by passing parameters as you
# would to find.
#
# Note this doesn't change file (timestamps)
# which don't need to be updated.

#Note super sed has a -i option to do this (edit files in place)
#also perl can edit files in place easily.

# Temporary file
temp=/tmp/runsed$$

find "$@" -type f -print |
while read file
do
    echo -n "editing $file: "
    if test -s $file; then
       sed -e 's/[ ]*$//g' <$file > $temp
       if test -s $temp; then
           if cmp -s $file $temp; then
             echo -n "file not changed: "
           else
             cp $temp $file
           fi
           echo "done"
       else
           echo "produced an empty file - aborting"
       fi
    else
       echo "original file is empty."
    fi
done
echo "all done"
rm -f $temp

-
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 : Sun Jun 30 2002 - 22:00:07 EST