<i18n dev>RFR: JDK-8144988: Unexpected timezone returned after parsing a date
masayoshi.okutsu at oracle.com
Mon Jan 18 06:47:36 UTC 2016
Looks good to me.
On 1/16/2016 12:31 AM, Ramanand Patil wrote:
> 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/
> -----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,
> 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.
> On 1/14/2016 9:21 PM, Ramanand Patil wrote:
>> Hi all,
>> Please review the fix for bug:
>> Webrev: http://cr.openjdk.java.net/~rpatil/8141243/webrev.00
>> This is basically a backport of JDK9 bug:
>> JDK9 changeset(for reference):
>> 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.
More information about the core-libs-dev