RFR: 8077587: BigInteger Roots [v52]
Raffaello Giulietti
rgiulietti at openjdk.org
Tue Jul 29 14:10:01 UTC 2025
On Tue, 29 Jul 2025 13:55:08 GMT, fabioromano1 <duke at openjdk.org> wrote:
>> Well, that was the easy case!
>> (BTW, the condition shExcess != 0 on L.2025 is always true, as shExcess > Double.MAX_EXPONENT - Double.PRECISION there.)
>>
>> There should be similar reasoning for the `else` part on L.2020.
>>
>> And, more importantly so, for the more obscure parts (for a 1st time reader, not for the author ;-) ) starting at L.2034. This is the code that took most of my time.
>
>> starting at L.2034. This is the code that took most of my time.
>
> @rgiulietti And at what line the obscure parts end?
Ah sorry, it ends at L.2073, with the end of the loop.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24898#discussion_r2239970773
More information about the core-libs-dev
mailing list