RFR: 8341402: BigDecimal's square root optimization [v20]
Raffaello Giulietti
rgiulietti at openjdk.org
Wed Nov 20 12:17:21 UTC 2024
On Tue, 19 Nov 2024 18:44:51 GMT, Raffaello Giulietti <rgiulietti at openjdk.org> wrote:
>> fabioromano1 has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Correct remainder checking
>
> src/java.base/share/classes/java/math/BigDecimal.java line 2260:
>
>> 2258: }
>> 2259:
>> 2260: default -> throw new AssertionError("Unexpected value for RoundingMode: " + mc.roundingMode);
>
> I think `MatchException` might be better suited (with a `null` cause), but I'll crosscheck and let you know tomorrow.
You can leave `AssertionError` as it is.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21301#discussion_r1850215392
More information about the core-libs-dev
mailing list