SplittableRandom update
Paul Sandoz
paul.sandoz at oracle.com
Mon Oct 7 13:24:06 UTC 2013
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 <dl at 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
More information about the core-libs-dev
mailing list