On Wed, 6 Aug 2025 18:56:15 GMT, Justin Lu <j...@openjdk.org> wrote:
>> Naoto Sato has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Refining docs
>
> src/java.base/share/classes/java/text/DecimalFormat.java line 310:
>
>> 308: * minimal digits, and other characteristics are all the same as the
>> positive
>> 309: * pattern. That means that {@code "#,##0.0#;(#)"} produces precisely
>> 310: * the same behavior as {@code "#,##0.0#;(#,##0.0#)"}. In
>
> I think it is appropriate to have this here since it regards the negative sub
> pattern. However, we should either link to this info or add additional info
> in the parse method. (Since the `parse` method covers behavior when
> strict/lenient.)
Since the leniency relates to not only `parse()` but also `is/setStrict()`
methods, I wanted to consolidate the minus sign leniency in the class
descrption. But yes describing it additionally in `parse()` would not hurt.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26580#discussion_r2258215992
