[OpenAFS-win32-devel] rawops.c

Simon Wilkinson sxw@inf.ed.ac.uk
Mon, 24 Aug 2009 22:31:38 +0100


On 24 Aug 2009, at 22:09, Matt W. Benjamin wrote:

> I wonder if the question may have been asked in the code-cleanup  
> spirit, as with many of Simon's recent (non-Windows) changes.

To date, I haven't removed any dead code as part of the code cleanup.

Where a function that is declared static is unused in the file  
containing it, I've #if 0'd it, rather than going for wholescale  
removal. I've not even looked at those functions which are exported  
but unused, of which I suspect we have a great number. There's  
obviously an issue of library size, but it's also worth considering  
that our source is our institutional memory. Some of these functions  
may provide ways of doing things that are unused at present, but  
useful in the future. Once they're gone from the 'current' code, it's  
unlikely that people will trawl back through the revision history on  
the off-chance of finding them.

S.