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

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

[Lightweight patch manager]

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?

This patch is available as
<URL:http://luckynet.dynu.com/~thunder/patches/double-quotes-1-rev.patch>
-- 
Lightweight patch manager using pine. If you have any objections, tell me.

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

[Pavel Machek]

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
-- 
(about SSSCA) "I don't say this lightly.  However, I really think that the U.S.
no longer is classifiable as a democracy, but rather as a plutocracy." --hpa

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

[Padraig Brady]

[-- Attachment #1: Type: text/plain, Size: 679 bytes --]

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.

[-- Attachment #2: rmws --]
[-- Type: text/plain, Size: 902 bytes --]

#!/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