<i18n dev> RFR: JDK8U Backport of 8171049: Era.getDisplayName doesn't work with non-IsoChronology

Deepak Kejriwal deepak.kejriwal at oracle.com
Wed Nov 7 12:09:28 UTC 2018


Hi Naoto,

Thanks for reviewing the fix. As you mentioned in review comment, I will add the copyright year 2018 in test case.

Regards,
Deepak

-----Original Message-----
From: Naoto Sato 
Sent: Monday, November 5, 2018 11:58 PM
To: Deepak Kejriwal <deepak.kejriwal at oracle.com>; core-libs-dev <core-libs-dev at openjdk.java.net>; i18n-dev at openjdk.java.net
Subject: Re: <i18n dev> RFR: JDK8U Backport of 8171049: Era.getDisplayName doesn't work with non-IsoChronology

Looks good. Since you modified the test case, please add the copyright year 2018 in the test case before you push.

Naoto

On 11/4/18 10:26 PM, Deepak Kejriwal wrote:
> Hi all,
> 
>   
> 
> Please review the fix for JDK8u Backport of 
> https://bugs.openjdk.java.net/browse/JDK-8171049
> 
> Webrev: http://cr.openjdk.java.net/~rpatil/8171049/webrev.00/
> 
>   
> 
> Master Bug changeset and Review details:
> 
> JDK Change set: 
> http://hg.openjdk.java.net/jdk10/jdk10/jdk/rev/965d4dde0086
> 
> JDK Review Thread: 
> http://mail.openjdk.java.net/pipermail/core-libs-dev/2017-August/04903
> 7.html
> 
>   
> 
> New test file "TestEraDisplayName.java" added as part of fix is 
> modified as per jdk 8 release:-
> 
>     
> 
> .         All test written in jdk 10 are written to verify the CLDR resource. Since, in case of jdk 8 "CLDR" is not default locale providers, added Djava.locale.providers=CLDR as JVM parameter to test.
> 
> .         Some resource keys in jdk 8 does not exist in both CLDR and JRE due to which Era.getDisplayName() returns numeric value. Modified such test cases accordingly.
> 
>   
> 
> Regards,
> 
> Deepak
> 


More information about the core-libs-dev mailing list