[13] RFR: 8220224: With CLDR provider, NumberFormat.format could not handle locale with number extension correctly.

Joseph D. Darcy joe.darcy at oracle.com
Thu Mar 21 21:11:38 UTC 2019

Hi Naoto,

Looks okay. The serial-related declarations are still using 
<code></code>; I think it is fine to push what you have now, but what 
encourage a follow-up bug to replace <code>foo</code> with {@code foo} 
throughout the entire class.



On 3/21/2019 1:54 PM, Naoto Sato wrote:
> Hello,
> Please review the fix to the following issue:
> https://bugs.openjdk.java.net/browse/JDK-8220224
> Here is the CSR and proposed changeset:
> https://bugs.openjdk.java.net/browse/JDK-8220728
> http://cr.openjdk.java.net/~naoto/8220224/webrev.01/
> DecimalFormatSymbols assumes minus/percent/permille as a single 
> character, which is not capable of supporting ones for BiDi languages 
> that involve BiDi formatting characters. With this fix, 
> DecimalFormatSymbols uses String variants of symbols from CLDR, and 
> retains them for serialization.
> The above webrev contains <code> tag cleanup which was suggested in 
> the CSR. The following webrev only contains relevant changes to the 
> issue (excluding those <code> cleanup):
> http://cr.openjdk.java.net/~naoto/8220224/webrev.00/
> Naoto

More information about the core-libs-dev mailing list