[loc-en-dev] New webrev available

Naoto Sato naoto.sato at oracle.com
Fri Sep 3 14:59:26 PDT 2010


Here are some more findings:

DecimalFormatSymbols.java
- No need to statically create locale_th_TH_TH instance, since it is 
rarely used. Just create it when needed.

Locale.java
- DISPLAY_SCRIPT case is missing in LocaleNameGetter.getObject().

ResourceBundle.java
- CandidateListCache.getDefaultList() may better be "static".

AsciiUtil.java
- CaseInsensitiveKey.equals() can have "if (o == this) return true;" for 
better performance.

InternalLocaleBuilder.java
- setExtensions() should not catch LocaleSyntaxException itself. Rather 
it should have "throws" clause in the method signature.

Naoto

(9/1/10 2:20 PM), Naoto Sato wrote:
> Here is the updated webrev based on the recent discussion:
>
> http://cr.openjdk.java.net/~naoto/6875847/webrev.01/
>
> Again, comments are welcome.
>
> Naoto



More information about the locale-enhancement-dev mailing list