On Wed, 25 Nov 2020 13:09:03 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 incrementally with one additional > commit since the last revision: > > 8248862: Implement Enhanced Pseudo-Random Number Generators > > Changes to RandomGeneratorFactory requested by @PaulSandoz src/java.base/share/classes/java/util/random/RandomGeneratorFactory.java line 88: > 86: * <pre>{@code > 87: * RandomGeneratorFactory<RandomGenerator> best = > RandomGenerator.all() > 88: * .sorted((f, g) -> Integer.compare(g.stateBits(), > f.stateBits())) use Comparator.comparingInt(RandomGenerator::stateBits) instead of the lambda src/java.base/share/classes/java/util/random/RandomGeneratorFactory.java line 116: > 114: * Map of properties for provider. > 115: */ > 116: private volatile Map<RandomGeneratorProperty, Object> properties = > null; `= null` is useless here src/java.base/share/classes/java/util/random/RandomGeneratorFactory.java line 106: > 104: * Map of provider classes. > 105: */ > 106: private static volatile Map<String, Provider<? extends > RandomGenerator>> factoryMap; should be FACTORY_MAP given that it's a static field ------------- PR: https://git.openjdk.java.net/jdk/pull/1292
