[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
lynx-dev Dead code removal (was: Re: why not --disable-news)
From: |
Kim DeVaughn |
Subject: |
lynx-dev Dead code removal (was: Re: why not --disable-news) |
Date: |
Mon, 8 Mar 1999 05:46:03 -0800 |
[OK ... Subject: changed :-) ]
On Mon, Mar 08, 1999, John Bley (address@hidden) said:
|
| On Sun, 7 Mar 1999, Kim DeVaughn wrote:
| >
| > | Meanwhile, I'm busy investigating dead code and have a question: Are all
| > | these empty HText_* functions here for any particular reason? Nothing's
| > | even calling into them. There are even several non-empty HText_
| > | functions that aren't ever getting called.
| >
| > Which ones ...?
|
| HText_canScrollUp for example. HText_dump, HText_nodeAnchor. There are
| about a dozen or so of these. Plus one LY method.. something to do with
| image maps. I should quit throwing away experimental files.
|
| > Are you sure they're not called (some are called from code in the
| > WWW/Lbrary/Implementations dir, BTW)?
|
| A grep HText_dump `find .` in the top-level dir yields (space truncated):
| src/GridText.c:PUBLIC void HText_dump ARGS1(
| WWW/Library/Implementation/HText.h:#define HText_dump HTHTDump
| WWW/Library/Implementation/HText.h:extern void HText_dump PARAMS((HText*me))
|
| I.e., the definition, a name-shortening #define, and a declaration of its
| existence. No call. This example is typical of the rest. I'll try to
| track down the complete list sometime soon. Perhaps Tuesday.
Be sure to check on the short names for usage, as well.
Also, as Tom pointed out, some fn()'s are called in obscure and circuitous
ways. I dunno if lynx ever calls fn()'s through pointers, but that is one
other thing to lookout for.
In short ... caveat emptor.
Also, how much in the way of object size would removing these potentially
useful fn()'s save, if in fact they really are unreferenced? For the sake
of "completeness" (by some metric), it may be better to leave them in.
/kim
- lynx-dev --disable-gopher, why not --disable-finger?, John Bley, 1999/03/05
- Re: lynx-dev --disable-gopher, why not --disable-finger?, Webmaster Jim, 1999/03/06
- Re: lynx-dev --disable-gopher, why not --disable-finger?, John Bley, 1999/03/06
- why not --disable--news (was: lynx-dev --disable-gopher, why not --disable-finger)?, Webmaster Jim, 1999/03/06
- Re: why not --disable--news (was: lynx-dev --disable-gopher, why not --disable-finger)?, Webmaster Jim, 1999/03/07
- lynx-dev Re: why not --disable-news, John Bley, 1999/03/07
- Re: lynx-dev why not --disable-news, Kim DeVaughn, 1999/03/07
- Re: lynx-dev why not --disable-news, John Bley, 1999/03/08
- lynx-dev Dead code removal (was: Re: why not --disable-news),
Kim DeVaughn <=
- Re: lynx-dev Dead code removal, John Bley, 1999/03/08
- Re: lynx-dev Dead code removal, Kim DeVaughn, 1999/03/08
- Re: lynx-dev Dead code removal, Philip Webb, 1999/03/08
- Re: lynx-dev Dead code removal, Klaus Weide, 1999/03/08
- Re: lynx-dev Dead code removal, Leonid Pauzner, 1999/03/09
- Re: lynx-dev why not --disable-news, mattack, 1999/03/08
- lynx-dev Re: dead code, John Bley, 1999/03/08
- lynx-dev re: why not --disable-news, Webmaster Jim, 1999/03/08
- lynx-dev Re: dead code removal, John Bley, 1999/03/08
- lynx-dev Q: Proxy Automatic Configuration, Klaus Peter Wegge, 1999/03/08