On Mon, 4 Jan 2021 19:52:18 GMT, Jim Laskey <jlas...@openjdk.org> wrote:

>> This PR is to introduce a new random number API for the JDK. The primary API 
>> is found in RandomGenerator and RandomGeneratorFactory. Further description 
>> can be found in the JEP https://openjdk.java.net/jeps/356 .
>> 
>> javadoc can be found at 
>> http://cr.openjdk.java.net/~jlaskey/prng/doc/api/java.base/java/util/random/package-summary.html
>> 
>> old PR:  https://github.com/openjdk/jdk/pull/1273
>
> 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)

-------------

PR: https://git.openjdk.java.net/jdk/pull/1292

Reply via email to