All of lore.kernel.org
 help / color / mirror / Atom feed
From: Padraig Brady <padraig@antefacto.com>
To: Mark Hahn <hahn@physics.mcmaster.ca>
Cc: linux-kernel@vger.kernel.org, Linus Torvalds <torvalds@transmeta.com>
Subject: Re: [PATCH] remove trailing whitespace
Date: Fri, 23 Nov 2001 20:34:36 +0000	[thread overview]
Message-ID: <3BFEB2DC.6040208@antefacto.com> (raw)
In-Reply-To: <Pine.LNX.4.10.10111231440260.1920-100000@coffee.psychology.mcmaster.ca>

I used sed, but yes the following does
the same as downloading and applying the patch:
find linux -type f | xargs perl -wi -pe 's/[<space><tab>]+$//'
(obviously replace <space> & <tab> with the appropriate chars).

Note also that after (bz2) compression the space saving drops
from 224,654 to 139,669 bytes, which is still good.

Padraig.

Mark Hahn wrote:

> 	find linux -type f | xargs perl -pe 's/\s+$/\n/'
> wouldn't something like this work?  seems generally easier,
> more compact, and probably more Linus-appetizing...
> 
> 
>>This (23MB! (5Mb compressed)) patch removes trailing whitespace from
>>all files in the kernel, thereby reducing size from 121,865,495 to
>>121,640,841. I.E. reducing size by 224,654 bytes. I don't know if it's
>>of any use, but it should be applied now if it is going to be done
>>at all.
>>
>>http://www.iol.ie/~padraiga/linux-2.5.0-strip-ws.diff.gz
>>
>>cheers,
>>Padraig.




       reply	other threads:[~2001-11-23 20:37 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <Pine.LNX.4.10.10111231440260.1920-100000@coffee.psychology.mcmaster.ca>
2001-11-23 20:34 ` Padraig Brady [this message]
2001-11-23 20:45   ` [PATCH] remove trailing whitespace David Weinehall
2001-11-23 17:20 Padraig Brady
2001-11-25 16:00 ` vda
2001-11-25 12:25   ` Russell King

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3BFEB2DC.6040208@antefacto.com \
    --to=padraig@antefacto.com \
    --cc=hahn@physics.mcmaster.ca \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@transmeta.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.