On Tue, 31 May 2022 17:46:18 GMT, Naoto Sato <na...@openjdk.org> wrote:
> Refactoring some old code in locale providers. The test case data have also > been modified due to: > - There's a bug in `LocaleProviderAdapter.toLocaleArray()` where it did not > handle the case for `no-NO-NY`. > - `Locale.toLanguageTag()` won't handle legacy Java locales, e.g., `ja_JP_JP` > and falls back, so comparing locales using language tags does not work for > those locales. Changed to compare with `Locale.toString()` instead. src/java.base/share/classes/sun/util/cldr/CLDRLocaleProviderAdapter.java line 181: > 179: .toArray(Locale[]::new); > 180: } > 181: return AVAILABLE_LOCALES; This should probably clone the cached array: Suggestion: return AVAILABLE_LOCALES.clone(); Matching what `JRELocaleProviderAdapter` does[^1], since there’s no guarantee that the result of `getAvailableLocales()` won’t be mutated. [^1]: https://github.com/openjdk/jdk/blob/6b1169e266b9d21864f886ef574dd64116fa2cb0/src/java.base/share/classes/sun/util/locale/provider/JRELocaleProviderAdapter.java#L430-L439 ------------- PR: https://git.openjdk.java.net/jdk/pull/8960