On Sun, 20 Apr 2025 16:07:56 GMT, fabioromano1 <d...@openjdk.org> wrote:
>> This PR implements nth root computation for `BigInteger`s using Newton >> method and optimizes `BigInteger.pow(int)` method. >> [Here is a proof of convergence of the recurrence >> used.](https://github.com/user-attachments/files/19785045/nth_root_newton_proof_integers.pdf) > > fabioromano1 has updated the pull request incrementally with one additional > commit since the last revision: > > Code simplification src/java.base/share/classes/java/math/MutableBigInteger.java line 1924: > 1922: * @implNote The implementation is based on the material in Henry > S. Warren, > 1923: * Jr., <i>Hacker's Delight (2nd ed.)</i> (Addison Wesley, 2013), > 279-282. > 1924: * * @implNote The implementation is based on the material in Henry S. Warren, * Jr., <i>Hacker's Delight (2nd ed.)</i> (Addison Wesley, 2013), 279-282. * I'm looking at this reference, and I only see integer square root here. But this is n >= 3. I do see an explanation at [nth root](https://en.wikipedia.org/wiki/Nth_root#Computing_principal_roots) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24690#discussion_r2052217018