Code Review 7051516: ThreadLocalRandom seed is never initialized so all instances generate the same sequence (original) (raw)

Mike Duigou mike.duigou at oracle.com
Thu Jun 16 16:32:49 UTC 2011

Hi Chris;

The getClass() seems unnecessary. Why not :

public Random(long seed) {
        this.seed = new AtomicLong();
        setSeed(seed);
}

or

private final AtomicLong seed = new AtomicLong();

public Random(long seed) { setSeed(seed); }

Both of these would seem to have the same effect without needing to do the explicit class check.

Mike

On Jun 16 2011, at 03:18 , Chris Hegarty wrote:

Hi,

ThreadLocalRandom uses its own seed, not the seed of the super class. ThreadLocalRandom() { super(); initialized = true; } This uselessly initializes the parent seed via the default constructor but leaves the real seed at zero. Webrev: http://cr.openjdk.java.net/~chegar/7051516/jdk8webrev.00/webrev/ Note: This is a port of the fix from Doug's CVS to OpenJDK. A test has been added to Doug's CVS tck tests for ThreadLocalRandom, and I will file a CR against the JCK to have it pulled in to JCK8. -Chris.

More information about the core-libs-dev mailing list