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