On Thu, 15 Apr 2021 12:17:46 GMT, Jim Laskey <jlas...@openjdk.org> wrote:

>> src/java.base/share/classes/jdk/internal/util/random/RandomSupport.java line 
>> 1656:
>> 
>>> 1654:         // Methods required by class AbstractSpliteratorGenerator
>>> 1655: 
>>> 1656:         protected Spliterator.OfInt makeIntsSpliterator(long index, 
>>> long fence, int origin, int bound) {
>> 
>> Are those still needed? It looks like this was not detected because of the 
>> missing `@Override`. Those could also be private or removed at all? The code 
>> is also handled by the above private versions (`this` is not instanceof 
>> `ThreadLocalRandom`).
>
> True that. There are also some mentions in the comments.

Thank you.

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

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

Reply via email to