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