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