RFR: 8077587: BigInteger Roots [v82]

fabioromano1 duke at openjdk.org
Mon Sep 8 15:16:20 UTC 2025


On Mon, 8 Sep 2025 15:08:24 GMT, Raffaello Giulietti <rgiulietti at openjdk.org> wrote:

>> fabioromano1 has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Revision changes
>
> Personally I tend to remove code that is not used.
> I can't recall why it was added: there might have been reasons that do no longer seem to be relevant.
> It should be straightforward to add it once need arises.

@rgiulietti The reason was that the `double` approximation of the root couldn't be casted directly into an `int` before taking into account to shift out the wrong bits.

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

PR Comment: https://git.openjdk.org/jdk/pull/24898#issuecomment-3266789071


More information about the core-libs-dev mailing list