RFR: 8248862: Implement Enhanced Pseudo-Random Number Generators [v9]
Jim Laskey
jlaskey at openjdk.java.net
Wed Jan 6 15:36:29 UTC 2021
On Tue, 5 Jan 2021 02:43:08 GMT, Brett Okken <github.com+2996845+bokken at openjdk.org> wrote:
>> Jim Laskey has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 34 commits:
>>
>> - Merge branch 'master' into 8248862
>> - 8248862: Implement Enhanced Pseudo-Random Number Generators
>>
>> Added coverage testing
>> - 8248862: Implement Enhanced Pseudo-Random Number Generators
>>
>> Cleanups from Chris H.
>> - 8248862: Implement Enhanced Pseudo-Random Number Generators
>>
>> Propagate exception
>> - 8248862: Implement Enhanced Pseudo-Random Number Generators
>>
>> Override AbstractSplittableGenerator
>> - 8248862: Implement Enhanced Pseudo-Random Number Generators
>>
>> Fix extends
>> - 8248862: Implement Enhanced Pseudo-Random Number Generators
>>
>> Use Map.of instead of Map.ofEntries
>> - 8248862: Implement Enhanced Pseudo-Random Number Generators
>>
>> Changes to RandomGeneratorFactory requested by @PaulSandoz
>> - Review changes
>>
>> @PaulSandoz and @rogermb
>> - 8248862: Implement Enhanced Pseudo-Random Number Generators
>>
>> Update package-info.java
>> - ... and 24 more: https://git.openjdk.java.net/jdk/compare/f80c63b3...e75cc844
>
> src/java.base/share/classes/java/util/random/RandomGenerator.java line 439:
>
>> 437: * Fills a user-supplied byte array with generated byte values
>> 438: * (pseudo)randomly chosen uniformly from the range of values between -128
>> 439: * (inclusive) and 255 (inclusive).
>
> Should this be
> between -128 (inclusive) and **127** (inclusive)
Thanks. Will update.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1292
More information about the core-libs-dev
mailing list