Extraneous whitespace removal?

["Jeremy M. Dolan" <jmd@foozle.turbogeek.org>]

Well, I'll let the number speak:

$ cp -R linux-2.4.0 linux-2.4.0-trimed
$ find linux-2.4.0-trimed -type f | xargs perl -wi -pe 's/\s+$/\n/'
$ du -s linux-2.4.0 linux-2.4.0-trimed
119360  linux-2.4.0              # NOTE: 4k blocks, just an estimate
119160  linux-2.4.0-trimed

$ diff -ru linux-2.4.0 linux-2.4.0-trimed > trimed.diff
$ ls -l trimed.diff*
-rw-r--r--   1 jmd      users    19131225 Jan  8 01:49 trimed.diff
-rw-r--r--   1 jmd      users     4732306 Jan  8 01:50 trimed.diff.gz
-rw-r--r--   1 jmd      users     3819235 Jan  8 01:52 trimed.diff.bz2

Pluses:
 - clean up messy whitespace
 - cut precious picoseconds off compile time
 - cut kernel tree by 200k (+/- alot)

Minuses:
 - adds 3.8M bzip2 or 4.7M gzip to next diff

Notes:
 - Don't actually use the above perl s// command. Instead, use
   [<tab><space>] in place of the \s. The problem with \s is it
   includes page breaks. I only included this one since the one with
   tab isn't cut&paste-able.
 - I'm not yet positive there are no other places in the tree that
   aren't safe to s/[<tab><space>]+$//. C can, if formated poorly
   enough, be affected by it (multiline strings not ending with \).
   Can anyone very familiar with Makefile/DocBook/TeX/asm syntax
   comment if they could also be potentially affected?
 - Another place to save space is extraneous \n's before EOF. I think
   that saves an aditional 15k or so, based on rough estimates with
   grep.
 - Yes, I am pretty pedantic to propose a 19M patch that doesn't *DO*
   anything.
 
-- 
Jeremy M. Dolan <jmd@turbogeek.org>
OpenPGP key = http://turbogeek.org/openpgp-key
OpenPGP fingerprint = 494C 7A6E 19FB 026A 1F52  E0D5 5C5D 6228 DC43 3DEE
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/