Re: [9] RFR 8038436: Re-examine the mechanism to determine available localedata and cldrdata

2014-08-22 Thread Naoto Sato
On 8/22/14, 3:56 PM, Mandy Chung wrote: A service config file can contain multiple provider implementation classes. JDI connector is one example: http://hg.openjdk.java.net/jdk9/dev/jdk/file/74078474d9bd/src/jdk.jdi/share/classes/META-INF/services/com.sun.jdi.connect.Connector There may be som

Re: [9] RFR 8038436: Re-examine the mechanism to determine available localedata and cldrdata

2014-08-22 Thread Mandy Chung
On 8/22/14 3:37 PM, Naoto Sato wrote: I wonder ifavailableLanguageTags andgetSupportedLocaleString should return a list or an array of String (see comment below). There are two provider implementations for sun.util.locale.provider.LocaleDataMetaInfo and two service config files as yo

Re: [9] RFR 8038436: Re-examine the mechanism to determine available localedata and cldrdata

2014-08-22 Thread Naoto Sato
Thank you Mandy for a quick review. Please see my comments below. On 8/22/14, 2:26 PM, Mandy Chung wrote: On 8/22/14 11:46 AM, Naoto Sato wrote: http://cr.openjdk.java.net/~naoto/8038436/webrev.3/ I skimmed on the patch and have a few initial comment/questions. JREENLocaleDataMetaInfo JRENo

Re: [9] RFR 8038436: Re-examine the mechanism to determine available localedata and cldrdata

2014-08-22 Thread Mandy Chung
On 8/22/14 11:46 AM, Naoto Sato wrote: http://cr.openjdk.java.net/~naoto/8038436/webrev.3/ I skimmed on the patch and have a few initial comment/questions. JREENLocaleDataMetaInfo JRENonENLocaleDataMetaInfo - are the lists of locale names generated previously? The long lines need to be

[9] RFR 8038436: Re-examine the mechanism to determine available localedata and cldrdata

2014-08-22 Thread Naoto Sato
Hello, Please review the changes for the following issue: https://bugs.openjdk.java.net/browse/JDK-8038436 The proposed changes are located at: http://cr.openjdk.java.net/~naoto/8038436/webrev.3/ The idea is to introduce an SPI so that supported locales are dynamically searched at runtime, n

Re: RFR: 8049343: (tz) Support tzdata2014f

2014-08-22 Thread Aleksej Efimov
Masayoshi, You're right that the "root names" should be changed as part of this update. The names were changed according to Australian official document here: http://australia.gov.au/about-australia/our-country/time The fixed version of the webrev can be found here: http://cr.openjdk.java.net/~