RFR: 8332086: Remove the usage of ServiceLoader in j.u.r.RandomGeneratorFactory [v4]

Jaikiran Pai jpai at openjdk.org
Wed May 15 06:40:02 UTC 2024


On Wed, 15 May 2024 06:30:35 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Raffaello Giulietti has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Replace SimpleImmutableEntry constructor with Map.entry() factory method.
>
> src/java.base/share/classes/java/util/random/package-info.java line 85:
> 
>> 83:  * Then one can choose a specific implementation by giving the name of a generator
>> 84:  * algorithm to the static method {@link RandomGenerator#of}, in which case no
>> 85:  * seed is specified by the caller:
> 
> Perhaps reword this to:
> 
> 
>> ... in which case a {@code RandomGenerator} is constructed without any seed value:

An already existing issue in the specification in this file is that, a few lines below, we note that:

> <p> There are three groups of random number generator algorithm provided in Java: the Legacy group, the LXM group, and the Xoroshiro/Xoshiro group.

The "three" groups is misleading I think, since both later in the table as well as the `group()` method implementation on `RandomGeneratorFactory`, we return four distinct values "Legacy", "LXM", "Xoroshiro" and "Xoshiro". Should we reword this part of the documentation to remove the mention of "three"?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19212#discussion_r1601022606


More information about the core-libs-dev mailing list