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

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

2016-11-06 Thread Nishit Jain
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 array which is then returned, so that it does not beha