On Wed, 11 Oct 2023 15:22:59 GMT, Brian Burkhalter <b...@openjdk.org> wrote:
>> test/jdk/jdk/internal/math/FloatingDecimal/TestFloatingDecimal.java line 115: >> >>> 113: } >>> 114: >>> 115: assertTrue(failures == 0); >> >> Suggestion: >> >> assertEquals(0, failures); >> >> Gives a better message because it explicitly knows both the expected value >> (1st arg) and the actual value (2nd arg). >> Similarly for other `assertTrue` usages below). > > Good suggestion! Thanks. I'll change it. So changed in 3f4fc63d3f1657caef467e2fa09d846d750dcad4. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16130#discussion_r1355250130
