Re: RFR: 8288173: JDK-8202449 fix causes conformance test failure : api/java_util/Random/RandomGenerator/NextFloat.html

2022-06-10 Thread Joe Darcy
On Fri, 10 Jun 2022 08:36:57 GMT, Raffaello Giulietti  
wrote:

> This fixes a bug introduced with JDK-8202449.

Changes look okay, but please do a follow-up fix to add some regression tests 
for this condition.

-

Marked as reviewed by darcy (Reviewer).

PR: https://git.openjdk.org/jdk/pull/9120


Re: RFR: 8288173: JDK-8202449 fix causes conformance test failure : api/java_util/Random/RandomGenerator/NextFloat.html

2022-06-10 Thread Brian Burkhalter
On Fri, 10 Jun 2022 08:36:57 GMT, Raffaello Giulietti  
wrote:

> This fixes a bug introduced with JDK-8202449.

Marked as reviewed by bpb (Reviewer).

-

PR: https://git.openjdk.org/jdk/pull/9120


Re: RFR: 8288173: JDK-8202449 fix causes conformance test failure : api/java_util/Random/RandomGenerator/NextFloat.html

2022-06-10 Thread Raffaello Giulietti
On Fri, 10 Jun 2022 08:36:57 GMT, Raffaello Giulietti  
wrote:

> This fixes a bug introduced with JDK-8202449.

The fix reverts an inadvertent "correction" sneaked into JDK-8202449, restoring 
previous (correct) behavior.

-

PR: https://git.openjdk.org/jdk/pull/9120


RFR: 8288173: JDK-8202449 fix causes conformance test failure : api/java_util/Random/RandomGenerator/NextFloat.html

2022-06-10 Thread Raffaello Giulietti
This fixes a bug introduced with JDK-8202449.

-

Commit messages:
 - 8288173: JDK-8202449 fix causes conformance test failure

Changes: https://git.openjdk.org/jdk/pull/9120/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk=9120=00
  Issue: https://bugs.openjdk.org/browse/JDK-8288173
  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/9120.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9120/head:pull/9120

PR: https://git.openjdk.org/jdk/pull/9120