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

Naoto Sato naoto.sato at oracle.com
Fri Aug 29 21:07:07 UTC 2014

I incorporated the suggestions from Mandy and Alan. Also one change 
since the last webrev was to revert the hard-coding of the supported 
locales list back to the one which dynamically generates the lists at 
the build time. I initially thought static listing of locales would be 
less complex as to the build script/makefile, but on second thought it's 
less evil than possible future regressions.

Please review:


On 8/28/14, 11:34 AM, Naoto Sato wrote:
> Thank you, Mandy, Masayoshi, and Alan for your comments. I revised the
> changes based on your suggestions as follows:
> http://cr.openjdk.java.net/~naoto/8038436/webrev.4/
> Here are the changes since webrev.3
> - CLDRLocaleProviderAdapter.java: modified to throw
> UnsupportedOperationException with the actual exception set to its
> "cause." More descriptive comment on it.
> - *ProviderImpl.java: Removed changes to them. Initial thought was to
> make them performant by changing the langtags to static, but it turned
> out that wasn't necessary.
> - JREENLocaleDataMetaInfo.java/JRENonENLocaleDataMetaInfo.java: Renamed
> to EnLocaleDataMetaInfo and NonEnLocaleDataMetaInfo respectively for
> readability. String constants are wrapped for readability as well. Used
> getOrDefault() for getSupportedLocaleString().
> - Added test cases for SecurityException and JRE's supported locales for
> each service.
> I'd appreciate it if someone in the build-dev could review the makefile
> changes.
> Naoto
> On 8/22/14, 11:46 AM, Naoto Sato wrote:
>> 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, not depending on the existence of
>> physical jar files.
>> I'd appreciate it if build folks could review the makefile related
>> changes, i18n forks to review locale framework files, and Mandy from
>> modularization point of view.
>> Naoto

More information about the i18n-dev mailing list