Integrated: 8282665: [REDO] ByteBufferTest.java: replace endless recursion with RuntimeException in void ck(double x, double y)

Emanuel Peter duke at openjdk.java.net
Wed Mar 9 09:03:07 UTC 2022


On Mon, 7 Mar 2022 08:41:18 GMT, Emanuel Peter <duke at openjdk.java.net> wrote:

> Until now, ck for doubles ignored the case when either x or y was NaN, it for example would let pass x=NaN and y=2.0.
> Further, instead of throwing an exception, it went into infinite recursion, with an eventual StackOverflowError.
> Now, we throw an exception that prints the values. I fixed the logic, and added an explanation.
> 
> All tests are passing.

This pull request has now been integrated.

Changeset: d07f7c76
Author:    Emanuel Peter <emanuel.peter at oracle.com>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/d07f7c76c5df1473bffa41f10a89ca1e21e001ef
Stats:     12 lines in 1 file changed: 9 ins; 0 del; 3 mod

8282665: [REDO] ByteBufferTest.java: replace endless recursion with RuntimeException in void ck(double x, double y)

Reviewed-by: psandoz, thartmann

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

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


More information about the hotspot-compiler-dev mailing list