Re: Review Request for JDK-8145136:Upgrade CLDR locale data

2016-05-29 Thread Yuka Kamiya
+1 On 2016/05/30 8:15, Masayoshi Okutsu wrote: Other than the copyright year things, all looks good to me. I don't think further review is required for copyright update. Masayoshi On 5/28/2016 3:25 AM, Naoto Sato wrote: Hi Rachna, Here are my comments to the webrev (I am assuming the tool t

Re: Review Request for JDK-8145136:Upgrade CLDR locale data

2016-05-29 Thread Masayoshi Okutsu
Other than the copyright year things, all looks good to me. I don't think further review is required for copyright update. Masayoshi On 5/28/2016 3:25 AM, Naoto Sato wrote: Hi Rachna, Here are my comments to the webrev (I am assuming the tool that extracts JavaTime*.java are working correctl

Re: Review Request for JDK-8145136:Upgrade CLDR locale data

2016-05-27 Thread Naoto Sato
Hi Rachna, Here are my comments to the webrev (I am assuming the tool that extracts JavaTime*.java are working correctly, i.e., have not reviewed the data itself): - (All JavaTime*.java files): Unicode copyright notice is outdated. It should include the one that has "1991-2016" as the copyri

Review Request for JDK-8145136:Upgrade CLDR locale data

2016-05-26 Thread Rachna Goel
Hi all, Please review fix for JDK-8145136. Bug: https://bugs.openjdk.java.net/browse/JDK-8145136 webrev: http://cr.openjdk.java.net/~nishjain/rachna/CLDR29/8145136/webrev.00/ Fix: Currently JDK supports CLDR V27 which is upgraded to latest version 29. For more info: http://cldr.unicode.org

Review Request for JDK-8145136 : Upgrade CLDR Locale Data

2016-03-24 Thread Rachna Goel
Hello all, Please review the fix for JDK-8145136. Bug : https://bugs.openjdk.java.net/browse/JDK-8145136 Webrev: http://cr.openjdk.java.net/~rgoel/cldr28/webrev.00/ Fix: Upgraded CLDR data from Version 27 to 28. More information at : http://cldr.unicode.org/ -- Thanks, Rachna