On Tue, 5 Jan 2021 02:43:08 GMT, Brett Okken <github.com+2996845+bok...@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