Re: RFR: 8169191: (tz) Support tzdata2016i

2016-11-07 Thread Masayoshi Okutsu
+1 On 11/8/2016 1:36 AM, Martin Buchholz wrote: Looks good to me! On Mon, Nov 7, 2016 at 2:43 AM, Ramanand Patil wrote: Hi all, Please review the latest TZDATA integration (tzdata2016i) to JDK9. Bug: https://bugs.openjdk.java.net/browse/JDK-8169191 Webrev: http://cr.openjdk.java.net/~rpatil/

Re: Review Request for JDK-8166994: Improve sun.util.locale.LocaleMatcher

2016-11-07 Thread Naoto Sato
+1 Naoto On 11/6/16 11:51 PM, Nishit Jain wrote: Hi, Please review the fix for JDK-8166994 Bug: https://bugs.openjdk.java.net/browse/JDK-8166994 Webrev: http://cr.openjdk.java.net/~nishjain/8166994/webrev.02/ Fix: Modified values of LocaleEquivalentMaps.multiEquivsMap is stored in a temp arr

Re: Review Request for JDK-8166994: Improve sun.util.locale.LocaleMatcher

2016-11-07 Thread Masayoshi Okutsu
Looks good to me. Masayoshi On 11/7/2016 4:51 PM, Nishit Jain wrote: Hi, Please review the fix for JDK-8166994 Bug: https://bugs.openjdk.java.net/browse/JDK-8166994 Webrev: http://cr.openjdk.java.net/~nishjain/8166994/webrev.02/ Fix: Modified values of LocaleEquivalentMaps.multiEquivsMap is

Re: RFR: 8169191: (tz) Support tzdata2016i

2016-11-07 Thread Martin Buchholz
Looks good to me! On Mon, Nov 7, 2016 at 2:43 AM, Ramanand Patil wrote: > Hi all, > Please review the latest TZDATA integration (tzdata2016i) to JDK9. > Bug: https://bugs.openjdk.java.net/browse/JDK-8169191 > Webrev: http://cr.openjdk.java.net/~rpatil/8169191/webrev.00/ > > All the TimeZone rela

RFR: 8169191: (tz) Support tzdata2016i

2016-11-07 Thread Ramanand Patil
Hi all, Please review the latest TZDATA integration (tzdata2016i) to JDK9. Bug: https://bugs.openjdk.java.net/browse/JDK-8169191 Webrev: http://cr.openjdk.java.net/~rpatil/8169191/webrev.00/ All the TimeZone related tests are passed after integration. Regards, Ramanand.