[OpenAFS-win32-devel] rawops.c

Jeffrey Altman jaltman@secure-endpoints.com
Mon, 24 Aug 2009 15:01:25 -0700


Matt W. Benjamin wrote:
> Hi Simon,
> 
> I wasn't trying to say anything about specific your changes to date.  Sometimes I would think simply removing code could be preferable--because if really want to look at how old releases worked, I will in fact look at old releases to find out.  I guess it depends.  On the other hand, it's nice to schedule things for possible future deletion, and it does reduce code size, so #if 0 seems the ideal thing to do on a first pass.
> 
> Matt


The problem with #if 0 is that doing so results in code that bit rots
since no one notices when the code would otherwise fail to compile.
Linkers are fairly smart these days and often discard unused code
in optimized builds.