On Mon, 9 Jun 2025 20:07:53 GMT, Naoto Sato <[email protected]> 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
