newnews y2k bug in 2.2.1

Forrest J. Cavalier III mibsoft at mibsoftware.com
Thu Oct 28 16:03:23 UTC 1999


I believe this was committed 8 January 1999 by tale.

Are saving a few CPU cycles so important?  Replace the
test and subtraction with:

   current->tm_year %= 100;

Better, since current->tm_year has a defined meaning already,
using it as a "work" variable is confusing. I'd prefer
to see the adjustment of tm_year dropped, and the conditional
clause which follows rewritten to be:

   	if (year <= (current->tm_year % 100))

....But I'm too lazy to patch and test it,...so no one has
to listen to what I say.


More information about the inn-workers mailing list