lkml.org 
[lkml]   [2001]   [Jan]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectExtraneous whitespace removal?
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/

\
 
 \ /
  Last update: 2005-03-22 12:52    [W:0.032 / U:15.732 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site