From: "Scott Preece" <sepreece@gmail.com>
To: "Robert P. J. Day" <rpjday@mindspring.com>
Cc: "Pádraig Brady" <P@draigbrady.com>,
"Matt LaPlante" <kernel1@cyberdogtech.com>,
linux-kernel@vger.kernel.org
Subject: Re: A little coding style nugget of joy
Date: Thu, 20 Sep 2007 09:04:05 -0500 [thread overview]
Message-ID: <7b69d1470709200704o73317ff2w345af46c5f427a4d@mail.gmail.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0709200610180.15327@localhost.localdomain>
On 9/20/07, Robert P. J. Day <rpjday@mindspring.com> wrote:
> On Thu, 20 Sep 2007, Pádraig Brady wrote:
>
> > Matt LaPlante wrote:
> > > Since everyone loves random statistics, here are a few gems to give you a break from your busy day:
> > >
> > > Number of lines in the 2.6.22 Linux kernel source that include one or more trailing whitespaces: 135209
> > > Bytes saved by removing said whitespace: 151809
> > > Lines in the (unified) diff: 455437
> > > Size of the diff: 15M
> > > People brave enough to submit the patch: ~0
> >
> > It's gradually getting better so:
> > http://lwn.net/2001/1129/a/whitespace.php3
>
> and you wouldn't *believe* how much space you can save by getting rid
> of all that annoying indentation. and don't even get me *started* on
> those comments ...
>
> rday
---
I think you're on to something here. If we stored the files with all
the non-meaningful whitespace (including non-meaningful newlines)
removed, not only would we save disk space, but we would also
eliminate significant amounts of developer time and LKML bandwidth
currently expended on arguing about formatting. Everybody could just
run things through indent with whatever formatting they preferred.
Might make diffs ugly, though...
scott
--
scott preece
prev parent reply other threads:[~2007-09-20 14:04 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-19 16:34 Matt LaPlante
2007-09-19 17:13 ` Andi Kleen
2007-09-19 21:22 ` Andy Lutomirski
2007-09-19 21:30 ` Andi Kleen
2007-09-19 21:39 ` Andrew Lutomirski
2007-09-20 9:20 ` Pádraig Brady
2007-09-20 10:11 ` Robert P. J. Day
2007-09-20 14:04 ` Scott Preece [this message]
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=7b69d1470709200704o73317ff2w345af46c5f427a4d@mail.gmail.com \
--to=sepreece@gmail.com \
--cc=P@draigbrady.com \
--cc=kernel1@cyberdogtech.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rpjday@mindspring.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®