On Tue, 20 Oct 2026 22:14:07 GMT, Justin Lu <[email protected]> wrote:

> Clarify that `CompactNumberFormat` formatting that routes through the "0" 
> special pattern never produces output with fractional digits. This is 
> accomplished by doing a rewrite of the formatting section which now mentions 
> it and also shows an example. I don't find it necessary to add a warning to 
> `setMinimumFractionDigits` itself since none of the CNF setters propagate to 
> the underlying `DecimalFormat`, which is fully derived by the decimal pattern 
> (minus the fractional digits).
> 
> A CSR is required, and will be updated once the wording is finalized.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Looks fine

src/java.base/share/classes/java/text/CompactNumberFormat.java line 108:

> 106:  * integer digits, prefix and/or suffix, but no fractional part. If 
> needed, users can use
> 107:  * {@link #setMinimumFractionDigits(int)} to support fractional digits 
> for compact number formatting.
> 108:  * For example, using the following patterns {@code {"", "", "", 0K, 
> ...}} with the {@link

It's in the original spec, but `0K` should be double-quoted.

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

Marked as reviewed by naoto (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/33482#pullrequestreview-5592839768
PR Review Comment: https://git.openjdk.org/jdk/pull/33482#discussion_r4331222144

Reply via email to