Looks good to me.

Naoto

On 1/14/16 4:21 AM, Ramanand Patil wrote:
Hi all,

Please review the fix for bug: https://bugs.openjdk.java.net/browse/JDK-8144988

Webrev: http://cr.openjdk.java.net/~rpatil/8141243/webrev.00

This is basically a backport of JDK9 bug: 
https://bugs.openjdk.java.net/browse/JDK-8141243

JDK9 changeset(for reference): 
http://hg.openjdk.java.net/jdk9/dev/jdk/rev/a1aa2671f281

Reason for the review request is because of:
i)     Changes present in ResourceBundleGenerator.java file.
ii)   The patch from JDK9 does not automatically apply as is after using 
unshuffle_patch script. Few paths are adjusted as per the jdk8.

Since CLDR became the default locale data in JDK9 leading incompatible behavior 
with prior releases, the relevant code in ResourceBundleGenerator is also 
backported in this patch.
Even though JDK8 has CLDR locale provider disabled by default, the code change is done to 
be on safer side for cases where CLDR may be supporting "UTC" ID in the future.


Regards,
Ramanand.

Reply via email to