On Fri, 26 Apr 2024 09:41:21 GMT, Raffaello Giulietti <rgiulie...@openjdk.org> wrote:
>> Move all random generators mandated in package `java.util.random` and >> currently implemented in module `jdk.random` to module `java.base`, and >> remove module `jdk.random`. > > Raffaello Giulietti has updated the pull request incrementally with one > additional commit since the last revision: > > Terminology changes. One thing that bothers me is that the use of ServiceLoader is specified as "Implementation Requirements". I wonder if we can check the original CSR and discussions on the PR to see if we can dig up more of the original intent. How useful is it to deploy with additional RandomGeneratorFactory implementations on the class path or module path? If a standard provider interface then I would have expected to see more in the javadoc and to see the `@uses` tags in the module declaration. ------------- PR Comment: https://git.openjdk.org/jdk/pull/18932#issuecomment-2079429549
