<i18n dev> RFR: 8262744: Formatter '%g' conversion uses wrong format for BigDecimal rounding up to limits [v2]
Ian Graves
igraves at openjdk.java.net
Fri Apr 16 16:14:42 UTC 2021
On Fri, 16 Apr 2021 14:26:58 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> Ian Graves has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Adding BigDecimal equivalents to tests
>
> src/java.base/share/classes/java/util/Formatter.java line 3826:
>
>> 3824: BigDecimal tenToTheNegFour = BigDecimal.valueOf(1, 4);
>> 3825: BigDecimal tenToThePrec = BigDecimal.valueOf(1, -prec);
>> 3826: value = value.round(new MathContext(prec));
>
> While you are in the area, how about inlining the creation of the tenToTheNegFour and tenToThePrec values.
> They are used only once and may not be used at all. They don't appear to be needed except for the comparisons.
Makes sense to me.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3363
More information about the i18n-dev
mailing list