Hi Kasper, Can you please log a bug [1] (and also note that ThreadLocal/Random does the same) ?
My preference would be to get this code in and fix the error reporting separately. Thanks, Paul. [1] https://bugs.openjdk.java.net/ On Sep 20, 2013, at 8:25 AM, Kasper Nielsen <kaspe...@gmail.com> wrote: > Hi, > > This is minor, but I hate it when bounds check does not include the > specified parameters. > Makes it much harder to track down a bug, if all you have is a stack trace. > > - Kasper > > > On Fri, 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<https://bugs.openjdk.java.net/browse/JDK-8025136> >> >> http://cr.openjdk.java.net/~**psandoz/tl/JDK-8025136-SR-** >> enhancements/webrev/<http://cr.openjdk.java.net/~psandoz/tl/JDK-8025136-SR-enhancements/webrev/> >> >> I think we need one more reviewer for it. >> >> -Doug >>