While having one last look at this Doug and I found some "oops how the heck did that happen" miscommits with the mix64/mix32/nextGamma methods that contained test code for measuring the impact of shifts:
For corrections see: http://gee.cs.oswego.edu/cgi-bin/viewcvs.cgi/jsr166/src/main/java/util/SplittableRandom.java?r1=1.23&r2=1.24 Updated patch: http://cr.openjdk.java.net/~psandoz/tl/JDK-8025136-SR-enhancements/webrev/ I will commit this tomorrow unless there are objections. Paul. On Sep 20, 2013, at 4:11 PM, Doug Lea <d...@cs.oswego.edu> wrote: > > In the course of writing up a report (coming soon) that includes > discussion of SplittableRandom, we had a chance to further > analyze and test things, resulting in a few small improvements. Plus > some internal renamings to better reflect intent. > Plus now with the same initial seed mechanics > discussed a few days ago for ThreadLocalRandom. > > Thanks to Paul Sandoz for creating webrevs: > > https://bugs.openjdk.java.net/browse/JDK-8025136 > > http://cr.openjdk.java.net/~psandoz/tl/JDK-8025136-SR-enhancements/webrev/ > > I think we need one more reviewer for it. > > -Doug
