Re: Fix for small leak in TimeZone_md.c

2015-09-10 Thread Volker Simonis
Hi Christoph, the change looks good. Thanks for doing this clean-up. I'll sponsor the change. Regards, Volker On Wed, Sep 9, 2015 at 3:08 PM, Langer, Christoph wrote: > Hi Dmitry, > > let me comment on your suggestions: > 663 it might be better to use str*r*chr > -> a TZ variable could look l

[9] RFR: 8134915: Improve performance of CLDRLocaleProviderAdapter.getCandidateLocales

2015-09-10 Thread Naoto Sato
Hello, Please review the changes for the following issue: https://bugs.openjdk.java.net/browse/JDK-8134915 Here is the proposed fix: http://cr.openjdk.java.net/~naoto/8134915/webrev.00/ It is basically replacing ResourceBundle based locale list with a simple hash map, and getting rid of runt

Re: [9] RFR: 8134915: Improve performance of CLDRLocaleProviderAdapter.getCandidateLocales

2015-09-10 Thread Masayoshi Okutsu
Looks good to me. Masayoshi On 9/11/2015 7:49 AM, Naoto Sato wrote: Hello, Please review the changes for the following issue: https://bugs.openjdk.java.net/browse/JDK-8134915 Here is the proposed fix: http://cr.openjdk.java.net/~naoto/8134915/webrev.00/ It is basically replacing ResourceBu