On Tue, 19 Nov 2024 09:21:08 GMT, Raffaello Giulietti <rgiulie...@openjdk.org> 
wrote:

> While you are at it, it would be useful to add one test for each of the paths 
> in the new algorithm (including the exception cases) in the existing 
> [SquareRootTests](https://github.com/openjdk/jdk/blob/master/test/jdk/java/math/BigDecimal/SquareRootTests.java)
>  This helps to identify future regressions when the code is later modified 
> for any reason.

I will add the path-based tests once the path tree has reached a stable 
version, in order to avoid changing tests if the path tree changes.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21301#discussion_r1848773886

Reply via email to