On Tue, 6 Apr 2021 16:46:06 GMT, Jim Laskey <jlas...@openjdk.org> wrote:

>> SecureRandom is stochastic and will become unpredictable over time (hence 
>> the intermittent nature).
>
> Jim Laskey has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Add bug number

Marked as reviewed by darcy (Reviewer).

test/jdk/java/util/Random/RandomTestBsi1999.java line 442:

> 440:             setRNG(factory.name());
> 441: 
> 442:             if (factory.name().equals("Random")) {

A string-switch on the currentRNG name might be better structured here.

test/jdk/java/util/Random/RandomTestBsi1999.java line 443:

> 441: 
> 442:             if (currentRNG.equals("SecureRandom")) {
> 443:                 // Bug: 8264791

It is generally not necessary to include bug ids in the text of changesets.

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

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

Reply via email to