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
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
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
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
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
+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