[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