Re: testcase failure java/util/Locale/Bug6989440.java

2011-10-05 Thread David Holmes
This might not be related to the CME problem, but here: public static LocaleServiceProviderPool getPool(ClassLocaleServiceProvider> providerClass) { LocaleServiceProviderPool pool = poolOfPools.get(providerClass); if (pool == null) { LocaleServiceProviderPool newP

Re: testcase failure java/util/Locale/Bug6989440.java

2011-10-05 Thread David Holmes
On 6/10/2011 2:58 AM, Alan Bateman wrote: Chris Hegarty wrote: Alan, Naoto, David I filed CR 7098100: java/util/Locale/Bug6989440.java fails intermittently. If you're ok with it please review the patch (below) and I can push it to the tl repo. Job done! I assume there is also some underlying

Re: testcase failure java/util/Locale/Bug6989440.java

2011-10-05 Thread Naoto Sato
I will look into this. Reopened the original CR. Naoto On 10/5/11 9:58 AM, Alan Bateman wrote: Chris Hegarty wrote: Alan, Naoto, David I filed CR 7098100: java/util/Locale/Bug6989440.java fails intermittently. If you're ok with it please review the patch (below) and I can push it to the tl r

Re: testcase failure java/util/Locale/Bug6989440.java

2011-10-05 Thread Alan Bateman
Chris Hegarty wrote: Alan, Naoto, David I filed CR 7098100: java/util/Locale/Bug6989440.java fails intermittently. If you're ok with it please review the patch (below) and I can push it to the tl repo. Job done! I assume there is also some underlying issue in the Locale code and this might

Re: testcase failure java/util/Locale/Bug6989440.java

2011-10-05 Thread Naoto Sato
Looks good to me. Thanks! Naoto On 10/5/11 8:42 AM, Chris Hegarty wrote: Alan, Naoto, David I filed CR 7098100: java/util/Locale/Bug6989440.java fails intermittently. If you're ok with it please review the patch (below) and I can push it to the tl repo. Job done! >: hg diff test/java/util/L

Re: testcase failure java/util/Locale/Bug6989440.java

2011-10-05 Thread Chris Hegarty
Alan, Naoto, David I filed CR 7098100: java/util/Locale/Bug6989440.java fails intermittently. If you're ok with it please review the patch (below) and I can push it to the tl repo. Job done! >: hg diff test/java/util/Locale/Bug6989440.java diff -r 24741fe639a8 test/java/util/Locale/Bug6989440

Re: testcase failure java/util/Locale/Bug6989440.java

2011-10-05 Thread Naoto Sato
Hi Chris, I appreciate it, and will fix the test case. Naoto On 10/5/11 7:07 AM, Chris Hegarty wrote: Alan, David, I noticed CR 7027061 was closed as 'not a defect'. Should I file a new CR to have the jtreg test fixed (join on all created threads)? The test will still exercise what it is supp

Re: testcase failure java/util/Locale/Bug6989440.java

2011-10-05 Thread Chris Hegarty
Alan, David, I noticed CR 7027061 was closed as 'not a defect'. Should I file a new CR to have the jtreg test fixed (join on all created threads)? The test will still exercise what it is supposed to, and the ConcurrentModificationException issue can be investigated at another time. This way w