On Tuesday 06 May 2008 02:58, Daniel Cheng wrote: > On Tue, May 6, 2008 at 8:47 AM, Matthew Toseland > <[EMAIL PROTECTED]> wrote: > > On Monday 05 May 2008 14:01, [EMAIL PROTECTED] wrote: > > > Author: j16sdiz > > > Date: 2008-05-05 13:01:55 +0000 (Mon, 05 May 2008) > > > New Revision: 19759 > > > > > > Modified: > > > trunk/freenet/src/freenet/support/NumberedItemComparator.java > > > Log: > > > use assert > > > > Base must be odd?? Explain? > > > > Also the class should be package private if you're going to make this sort of > > assumptions... > > > > I didn't make up that comment, I just moved the comment from below. > Can not understand that either, that's why I added an "assert".
Well it probably makes more sense to implement the correct behaviour and remove the assert. > > > > + // REDFLAG: base must be odd, so we never get == > [...] > > > return -1; // i2>i1 > > > - // REDFLAG: base must be odd, so we never get ==
pgp4A6VYpNUUL.pgp
Description: PGP signature
_______________________________________________ Devl mailing list Devl@freenetproject.org http://emu.freenetproject.org/cgi-bin/mailman/listinfo/devl