<i18n dev> RFR: JDK-8144988: Unexpected timezone returned after parsing a date

[Ramanand Patil]

Hi Masayoshi,

Thank you for pointing that out. I have removed line 29 from the test.

Please review the updated Webrev: http://cr.openjdk.java.net/~rpatil/8141243/webrev.01/


Regards,
Ramanand.

-----Original Message-----
From: Masayoshi Okutsu 
Sent: Friday, January 15, 2016 8:18 AM
To: Ramanand Patil; i18n-dev at openjdk.java.net
Cc: core-libs-dev at openjdk.java.net
Subject: Re: <i18n dev>RFR: JDK-8144988: Unexpected timezone returned after parsing a date

Hi Ramanand,

test/java/text/Format/DateFormat/Bug8141243.java:

   28  * @run main Bug8141243
   29  * @run main/othervm -Djava.locale.providers=COMPAT Bug8141243

"COMPAT" is a new name of "JRE" in JDK 9. It's not supported in 8u. I think COMPAT is slightly ignored and that it becomes the same thing as line 28. Line 29 should be removed.

Otherwise, the fix looks OK to me.

Masayoshi

On 1/14/2016 9:21 PM, 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.