Re: [9] RFR: 8008577: Use CLDR Locale Data by Default (JEP-252)

2015-06-24 Thread Masayoshi Okutsu
applyParentLocales() sets parentLocalesMap before populating the map with data. It's possible that other threads look up the map without the (full) data. So, a Map (local variable) should be populated and then parentLocalesMap should be set to the Map. Also, parentLocalesMap needs to be volatil

RFR: 8098547: (tz) Support tzdata2015e

2015-06-24 Thread Aleksej Efimov
Hello, Please, review the latest tzdata (2015e) [1] integration to JDK9: http://cr.openjdk.java.net/~aefimov/tzdata/2015e/9/0 Testing shows no TZ related failures on all platforms. With Best Regards, Aleksej [1] https://bugs.openjdk.java.net/browse/JDK-8098547

Re: RFR: 8098547: (tz) Support tzdata2015e

2015-06-24 Thread Seán Coffey
Looks fine. Regards, Sean. On 24/06/15 12:05, Aleksej Efimov wrote: Hello, Please, review the latest tzdata (2015e) [1] integration to JDK9: http://cr.openjdk.java.net/~aefimov/tzdata/2015e/9/0 Testing shows no TZ related failures on all platforms. With Best Regards, Aleksej [1] https://bu

Re: [9] RFR: 8008577: Use CLDR Locale Data by Default (JEP-252)

2015-06-24 Thread Naoto Sato
Thanks. Here is the diff from "webrev.01" to address your comment: http://hg.openjdk.java.net/jdk9/sandbox/jdk/rev/b8faab65bb62 Naoto On 6/24/15 2:16 AM, Masayoshi Okutsu wrote: applyParentLocales() sets parentLocalesMap before populating the map with data. It's possible that other threads loo

Re: [9] RFR: 8008577: Use CLDR Locale Data by Default (JEP-252)

2015-06-24 Thread Masayoshi Okutsu
Looks good to me. Masayoshi On 6/25/2015 1:15 AM, Naoto Sato wrote: Thanks. Here is the diff from "webrev.01" to address your comment: http://hg.openjdk.java.net/jdk9/sandbox/jdk/rev/b8faab65bb62 Naoto On 6/24/15 2:16 AM, Masayoshi Okutsu wrote: applyParentLocales() sets parentLocalesMap be

Re: RFR: 8098547: (tz) Support tzdata2015e

2015-06-24 Thread Masayoshi Okutsu
+1 Masayoshi On 6/24/2015 9:58 PM, Seán Coffey wrote: Looks fine. Regards, Sean. On 24/06/15 12:05, Aleksej Efimov wrote: Hello, Please, review the latest tzdata (2015e) [1] integration to JDK9: http://cr.openjdk.java.net/~aefimov/tzdata/2015e/9/0 Testing shows no TZ related failures on a