RFR: 8329222: java.text.NumberFormat (and subclasses) spec updates [v2]
Naoto Sato
naoto at openjdk.org
Tue Apr 16 21:38:04 UTC 2024
On Tue, 16 Apr 2024 17:06:50 GMT, Justin Lu <jlu at openjdk.org> wrote:
>> Justin Lu has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits:
>>
>> - merge master and add setStrict() to nFmt class spec
>> - implement suggestions from dFmt review
>> - implement suggestions from cnFmt review
>> - implement suggestions from nFmt review
>> - init
>
> src/java.base/share/classes/java/text/NumberFormat.java line 115:
>
>> 113: * attributes or key/type pairs.
>> 114: * <p>Below is an example of a "US" locale currency format with accounting style,
>> 115: * <blockquote>{@code NumberFormat.getInstance(Locale.forLanguageTag("en-US-u-cf-account"));}</blockquote>
>
> I replaced the previous Thai digits example with an account currency format style, as it is probably a more common use case.
Good. I think adding the expectation would be more helpful, such as, "accounting style, that is, a negative value is enclosed in parentheses instead of a prepended minus sign."
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18731#discussion_r1567934872
More information about the core-libs-dev
mailing list