On Wed, 3 Sep 2025 09:24:32 GMT, Raffaello Giulietti <rgiulie...@openjdk.org> wrote:
>> fabioromano1 has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Revision changes > > src/java.base/share/classes/java/math/MutableBigInteger.java line 2059: > >> 2057: /* The Newton's recurrence roughly duplicates the >> correct bits at each iteration. >> 2058: * Instead of shifting the approximate root into the >> original range right now, >> 2059: * we only duplicate its bit length and then refine it >> with Newton's recurrence, > > Suggestion: > > /* The Newton's recurrence roughly doubles the correct bits > at each iteration. > * Instead of shifting the approximate root into the original > range right now, > * we only double its bit length and then refine it with > Newton's recurrence, In `BigIntegerTest`, the methods related to nth-root do not seem to be invoked. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24898#discussion_r2318395246
