<i18n dev> RFR: 8358734: Remove JavaTimeSupplementary resource bundles [v2]
Justin Lu
jlu at openjdk.org
Mon Jun 9 23:11:28 UTC 2025
On Mon, 9 Jun 2025 20:07:53 GMT, Naoto Sato <naoto at openjdk.org> wrote:
>> The parallel loading of JavaTimeSupplementary was a historical artifact from the introduction of JSR 310, which additionally loads resouces that had not existed before. Since the COMPAT locale provider which relied on this mechanism has been removed, and the CLDR resource bundles now include those resources by default, removing the parallel loading mechanism simplifies the implementation
>
> Naoto Sato has updated the pull request incrementally with one additional commit since the last revision:
>
> Removed the bundle
Cleanup looks good to me.
src/java.base/share/classes/sun/util/resources/LocaleData.java line 214:
> 212: int lastDot = baseName.lastIndexOf('.');
> 213: String category = (lastDot >= 0) ? baseName.substring(lastDot + 1) : baseName;
> 214: if (adapter instanceof JRELocaleProviderAdapter jlpa) {
For safety, should we have some type of failure if this is not true, similar to the previous `CCE`.
-------------
Marked as reviewed by jlu (Committer).
PR Review: https://git.openjdk.org/jdk/pull/25699#pullrequestreview-2911552652
PR Review Comment: https://git.openjdk.org/jdk/pull/25699#discussion_r2136628185
More information about the i18n-dev
mailing list