From: Frank van Maarseveen <frankvm@xs4all.nl>
To: Andries Brouwer <aebr@win.tue.nl>
Cc: Adrian Bunk <bunk@stusta.de>, Linus Torvalds <torvalds@osdl.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: bloat
Date: Sun, 7 Nov 2004 14:37:11 +0100 [thread overview]
Message-ID: <20041107133711.GA3310@janus> (raw)
In-Reply-To: <20041106214147.GA9663@pclin040.win.tue.nl>
On Sat, Nov 06, 2004 at 10:41:47PM +0100, Andries Brouwer wrote:
>
> I have one or two unused functions inside #if 0 in sddr09.c.
> Finding out the proper hardware details was nontrivial,
> it would be a pity to throw the knowledge away.
Of course nobody likes to throw away hard work. However, code that isn't
used tends to be unmaintained. So it might be outdated when (and if)
it gets used in the future. A web page indexed by search engines is a
excellent way to preserve knowledge.
The existence of unused code in the kernel will probably make not much
of a difference for future development. But it pollutes the source so
IMHO we could do without many of these:
$ find -name '*.[ch]' |xargs grep '^#if 0'|wc
2491 7202 97634 (2.6.9-rc3)
--
Frank
next prev parent reply other threads:[~2004-11-07 13:37 UTC|newest]
Thread overview: 71+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-11-03 21:02 support of older compilers Timothy Miller
2004-11-03 21:13 ` Christoph Hellwig
2004-11-03 21:22 ` Chris Wedgwood
2004-11-03 23:06 ` Adam Heath
2004-11-03 23:30 ` Chris Wedgwood
2004-11-04 16:50 ` Adam Heath
2004-11-04 17:00 ` Chris Friesen
2004-11-04 18:17 ` Adam Heath
2004-11-05 20:00 ` Willy Tarreau
2004-11-05 20:28 ` Geert Uytterhoeven
2004-11-05 20:31 ` Willy Tarreau
2004-11-04 17:04 ` Valdis.Kletnieks
2004-11-04 18:15 ` Adam Heath
2004-11-04 18:31 ` Valdis.Kletnieks
2004-11-04 21:56 ` Adam Heath
2004-11-04 18:54 ` Ian Romanick
2004-11-04 19:48 ` Christoph Hellwig
2004-11-04 20:14 ` Giuseppe Bilotta
2004-11-05 20:04 ` Willy Tarreau
2004-11-04 19:36 ` Ian Hastie
2004-11-04 20:02 ` Ioan Ionita
2004-11-04 20:03 ` Adrian Bunk
2004-11-04 20:08 ` Valdis.Kletnieks
2004-11-04 19:38 ` Linus Torvalds
2004-11-04 21:47 ` Adam Heath
2004-11-04 21:55 ` Linus Torvalds
2004-11-04 23:39 ` Adam Heath
2004-11-04 23:52 ` Linus Torvalds
2004-11-05 1:41 ` Andries Brouwer
2004-11-05 15:41 ` Linus Torvalds
2004-11-05 15:47 ` Arjan van de Ven
2004-11-05 16:22 ` linux-os
2004-11-05 19:50 ` Chris Wedgwood
2004-11-05 20:28 ` Linus Torvalds
2004-11-05 21:59 ` Grzegorz Kulewski
2004-11-05 22:08 ` Linus Torvalds
2004-11-05 23:02 ` [PATCH] change Kconfig entry for RAMFS (Was: Re: support of older compilers) Grzegorz Kulewski
2004-11-05 23:12 ` Linus Torvalds
2004-11-05 23:44 ` [PATCH] change Kconfig entry for RAMFS Grzegorz Kulewski
2004-11-06 0:48 ` Grzegorz Kulewski
2004-11-09 23:24 ` Matt Mackall
2004-11-10 10:40 ` Grzegorz Kulewski
2004-11-10 20:50 ` Bill Davidsen
2004-11-05 23:27 ` [PATCH] change Kconfig entry for RAMFS (Was: Re: support of older compilers) DaMouse
2004-11-05 23:49 ` Grzegorz Kulewski
2004-11-06 13:59 ` DaMouse
2004-11-05 22:08 ` support of older compilers Hua Zhong
2004-11-06 8:38 ` Willy Tarreau
2004-11-06 9:43 ` Hugh Dickins
2004-11-06 11:04 ` Willy Tarreau
2004-11-05 22:37 ` support of older compilers [u] Martin Schlemmer [c]
2004-11-05 23:14 ` Grzegorz Kulewski
2004-11-05 23:38 ` chibiryuu
2004-11-06 12:07 ` support of older compilers Andries Brouwer
2004-11-06 17:33 ` Linus Torvalds
2004-11-06 19:36 ` Adrian Bunk
2004-11-06 21:41 ` bloat Andries Brouwer
2004-11-06 22:07 ` bloat Adrian Bunk
2004-11-06 21:30 ` bloat Arnaldo Carvalho de Melo
2004-11-07 13:37 ` Frank van Maarseveen [this message]
2004-11-05 20:20 ` support of older compilers Willy Tarreau
2004-11-05 22:19 ` Adam Heath
2004-11-04 22:36 ` Martin J. Bligh
2004-11-04 20:48 ` Bill Davidsen
2004-11-04 22:33 ` Martin J. Bligh
2004-11-03 21:17 ` Matti Aarnio
2004-11-03 21:37 ` Giacomo A. Catenazzi
2004-11-03 21:57 ` Valdis.Kletnieks
2004-11-04 2:16 ` Miles Bader
2004-11-03 22:07 ` Chris Wedgwood
-- strict thread matches above, loose matches on Subject: below --
2011-08-21 22:58 bloat Anca Emanuel
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=20041107133711.GA3310@janus \
--to=frankvm@xs4all.nl \
--cc=aebr@win.tue.nl \
--cc=bunk@stusta.de \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@osdl.org \
/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