<i18n dev> RFR: 8151431: DateFormatSymbols triggers this.clone() in the constructor
masayoshi.okutsu at oracle.com
Thu Apr 7 05:09:32 UTC 2016
On 4/7/2016 2:44 AM, Naoto Sato wrote:
> Looks good overall. Here are my suggestions.
Thank you for your quick review.
> - DateFormatSymbols.java: We could cache all candidate locales, e.g.,
> if the requested locale is ja_JP_JP, and the bundle locale is ja, we
> could also put ja_JP bundle in the cache.
Right. But that one is too tricky to handle there.
> - DateFormatSymbolsCloneTest.java: import sentence for the Locale
> class can be removed.
Good catch. Fixed.
> On 4/6/16 2:10 AM, Masayoshi Okutsu wrote:
>> Hi all,
>> Please review the fix for JDK-8151431. clone() is no longer called to
>> create a cache entry.
More information about the i18n-dev