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

[Paul Sandoz]

On Mon, 7 Mar 2022 10:25:47 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.
>
> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
> 
>   state reason for failure in Exception

I wonder if we can simplify and replace the test with `Double.compare(x, y) != 0`? Can you test that?

That introduces a new condition since `-0.0 == 0.0` is `true` (yes, another odd difference!) where as `Double.compare(-0.0, 0.0) != 0`

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

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