On Monday 05 May 2008 14:01, j16sdiz at freenetproject.org 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...
>
>
> Modified: trunk/freenet/src/freenet/support/NumberedItemComparator.java
> ===================================================================
> --- trunk/freenet/src/freenet/support/NumberedItemComparator.java
> 2008-05-05
13:01:35 UTC (rev 19758)
> +++ trunk/freenet/src/freenet/support/NumberedItemComparator.java
> 2008-05-05
13:01:55 UTC (rev 19759)
> @@ -42,11 +42,13 @@
> secondDistance = i2 - i1; // smaller => i2 > i1
> firstDistance = i1 + Long.MAX_VALUE - i2; // smaller => i1
> i2
> }
> + // REDFLAG: base must be odd, so we never get ==
> + assert firstDistance != secondDistance &&
firstDistance != -secondDistance;
> +
> if(Math.abs(firstDistance) < Math.abs(secondDistance)) {
> return 1; // i1>i2
> } else //if(Math.abs(secondDistance) < Math.abs(firstDistance))
{
> return -1; // i2>i1
> - // REDFLAG: base must be odd, so we never get ==
> }
> }
>
>
> _______________________________________________
> cvs mailing list
> cvs at freenetproject.org
> http://emu.freenetproject.org/cgi-bin/mailman/listinfo/cvs
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL:
<https://emu.freenetproject.org/pipermail/devl/attachments/20080506/7f700e35/attachment.pgp>
