echelon/i2p.i2p
1
0
Fork 0
forked from I2P_Developers/i2p.i2p
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

bad jrandom, test first

Browse Source
This commit is contained in:
jrandom
2004-08-08 01:34:42 +00:00
committed by zzz
parent 2cb519cd06
commit 8d7786e97d
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
core/java/src/net/i2p/util/RandomSource.java
View File

@ -33,15 +33,15 @@ public class RandomSource extends SecureRandom {
* According to the java docs (http://java.sun.com/j2se/1.4.1/docs/api/java/util/Random.html#nextInt(int))
* nextInt(n) should return a number between 0 and n (including 0 and excluding n). However, their pseudocode,
* as well as sun's, kaffe's, and classpath's implementation INCLUDES NEGATIVE VALUES.
* WTF. Ok, so we're going to have it return between 0 and n (including both 0 and n), since
* WTF. Ok, so we're going to have it return between 0 and n, since
* thats what it has been used for.
*
*/
public int nextInt(int n) {
if (n == 0) return 0;
int val = super.nextInt(n+1);
int val = super.nextInt(n);
if (val < 0) val = 0 - val;
if (val > n) val = val % (n+1);
if (val > n) val = val % n;
return val;
}
@ -52,7 +52,7 @@ public class RandomSource extends SecureRandom {
public long nextLong(long n) {
long v = super.nextLong();
if (v < 0) v = 0 - v;
if (v > n) v = v % (n+1);
if (v > n) v = v % n;
return v;
}