Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261586AbUKGNhZ (ORCPT ); Sun, 7 Nov 2004 08:37:25 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261615AbUKGNhZ (ORCPT ); Sun, 7 Nov 2004 08:37:25 -0500 Received: from frankvm.xs4all.nl ([80.126.170.174]:23691 "EHLO janus.localdomain") by vger.kernel.org with ESMTP id S261586AbUKGNhN (ORCPT ); Sun, 7 Nov 2004 08:37:13 -0500 Date: Sun, 7 Nov 2004 14:37:11 +0100 From: Frank van Maarseveen To: Andries Brouwer Cc: Adrian Bunk , Linus Torvalds , Linux Kernel Mailing List Subject: Re: bloat Message-ID: <20041107133711.GA3310@janus> References: <20041105014146.GA7397@pclin040.win.tue.nl> <20041106120716.GA9144@pclin040.win.tue.nl> <20041106193605.GL1295@stusta.de> <20041106214147.GA9663@pclin040.win.tue.nl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20041106214147.GA9663@pclin040.win.tue.nl> User-Agent: Mutt/1.4.1i X-Subliminal-Message: Use Linux! Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1013 Lines: 25 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 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/