[8]Review request: 8001231 : Move locale data out of rt.jar (except the US locale)

2012-10-30 Thread Naoto Sato
Hello, I need someone to review my changes to the following bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8001231 The fix is to move locale data out of rt.jar into localedata.jar which will be optional in the compact profile (http://openjdk.java.net/jeps/161) The gist of the changes

[8]Request for review: 8001205) Calendar.getDisplayName(...): Returns null when provider is SPI but there is no SPI implementation

2012-10-30 Thread Naoto Sato
Hello, Please review the fix for the following bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8001205 The fix is simply to supply the fallback locale provider adapter for the root locale, when there is no explicit "JRE" adapter is specified in the locale provider preference list. T

[8]Review request for 8001440:, CLDR adapter: Invalid number extension in language tag causes exception in NumberFormat.format()

2012-10-30 Thread Naoto Sato
Here is another simple fix review request. The bug description can be found at: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8001440 The reason of the issue was that the "DefaultNumberingSystem" value was not retrieved correctly when the Unicode extension key was a bogus value. The prop

Re: [8]Request for review: 8001205) Calendar.getDisplayName(...): Returns null when provider is SPI but there is no SPI implementation

2012-10-30 Thread Masayoshi Okutsu
Question. Does the full set of available locales of JRE still become available with this fix? For example, when java.locale.providers=SPI, all JRE locales are included in the getAvailableLocales (API)? The fallback adapter is an interesting idea, but it's a bit confusing... Thanks, Masayoshi

Re: [8]Request for review: 8001205) Calendar.getDisplayName(...): Returns null when provider is SPI but there is no SPI implementation

2012-10-30 Thread Naoto Sato
With this fix, no. But another bug was filed regarding the locale sensitive service APIs' (e.g., Collator) getAvailableLocales() return value (8001562). I will fix it to keep the returned locale array compatible with JDK7. Naoto On 10/30/12 7:43 PM, Masayoshi Okutsu wrote: Question. Does the