<i18n dev> RFR: 8362448: Make use of the Double.toString(double) algorithm in java.text.DecimalFormat [v2]

[Raffaello Giulietti]

On Thu, 17 Jul 2025 17:58:51 GMT, Justin Lu <jlu at openjdk.org> wrote:

>> Raffaello Giulietti has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Added comment to COMPAT static field.
>
> src/java.base/share/classes/jdk/internal/math/FloatingDecimal.java line 1769:
> 
>> 1767:      * fields (> 550 lines).
>> 1768:      */
>> 1769:     private static BinaryToASCIIConverter getCompatBinaryToASCIIConverter(double d, boolean isCompatibleFormat) {
> 
> Can we just remove the `isCompatibleFormat` param and pass true to `buf.dtoa` which should always be true as indicated by the method name and its usage?

There are many more changes that I've planned in `FloatingDecimal`, including this one and the removal of unused methods.
For this PR, however, I just wanted to keep the changes in `FloatingDecimal` minimal, focused on the replacement with the new algorithm.

But I have no problems with your proposal, if you insist.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26364#discussion_r2214084343