From: | The Hermit Hacker <scrappy(at)hub(dot)org> |
---|---|
To: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers(at)postgreSQL(dot)org |
Subject: | Re: [HACKERS] freefuncs.c is never called from anywhere!? |
Date: | 2000-02-01 02:03:41 |
Message-ID: | Pine.BSF.4.21.0001312202400.411-100000@thelab.hub.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, 31 Jan 2000, Bruce Momjian wrote:
> > I think it would be a bad idea to go around and yank out NOT_USED code
> > wholesale --- this should be looked at on a case-by-case basis, with
> > consideration of whether the code might be useful again in future.
> >
> > But I think I might vote for pruning the _deadcode directories
> > wholesale. I suspect the files in them are suffering from
> > irrecoverable bit rot.
>
> I disagree on _deadcode. While the code is rotted, it does implement
> some full functions that have no other history, like verion.c for
> versioning and xfunc for expensive functions. Yanking them means we can
> never know what they did.
even 'yanked' code is still in the cvs repository ... is it something that
ppl actually refer to, or are just afraid of possibly needing/wanting to
go back and look at it? maybe just add a '.readme' file to a subdirectory
to note files that were removed or something like that?
Marc G. Fournier ICQ#7615664 IRC Nick: Scrappy
Systems Administrator @ hub.org
primary: scrappy(at)hub(dot)org secondary: scrappy(at){freebsd|postgresql}.org
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2000-02-01 02:04:31 | Re: [HACKERS] float4 confused as int?? |
Previous Message | Tom Lane | 2000-02-01 02:02:30 | Re: [HACKERS] Another nasty cache problem |