Please check...ignore them...

Started by The Hermit Hackerabout 27 years ago2 messages
#1The Hermit Hacker
scrappy@hub.org

Everything is in CVS, if I oops, its recoverable...am removing all code
that has either:

#if 0

or

#ifdef FALSE

surrounding it, since it isn't used...we have the logs and diffs in the
CVS repository if I make a mistake, but since it isn't used, can't see how
I can make a mistake :)

Marc G. Fournier
Systems Administrator @ hub.org
primary: scrappy@hub.org secondary: scrappy@{freebsd|postgresql}.org

#2Bruce Momjian
maillist@candle.pha.pa.us
In reply to: The Hermit Hacker (#1)
Re: [HACKERS] Please check...ignore them...

Everything is in CVS, if I oops, its recoverable...am removing all code
that has either:

#if 0

or

#ifdef FALSE

surrounding it, since it isn't used...we have the logs and diffs in the
CVS repository if I make a mistake, but since it isn't used, can't see how
I can make a mistake :)

Yep, except there is some code if'defed out that could be useful some
day. The one from dbcommand.c clearly would never be useful.
Basically, if you can figure out what the ifdef 0 did, and you see a
replacement in the code, you can remove it. That is the guide I use.

-- 
  Bruce Momjian                        |  http://www.op.net/~candle
  maillist@candle.pha.pa.us            |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026