On Wed, 23 Jul 2025 16:48:13 GMT, Justin Lu <[email protected]> wrote:
>> Naoto Sato has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Obsolete comment correction
>
> src/java.base/share/classes/java/util/Locale.java line 534:
>
>> 532: * their earlier, obsoleted forms: {@code he} maps to {@code iw},
>> 533: * {@code yi} maps to {@code ji}, and {@code id} maps to
>> 534: * {@code in}. Since Java SE 17, this is no longer the case. Each
>
> * <p>Locale's constructors have always converted three language codes to
> * their earlier, obsoleted forms: {@code he} maps to {@code iw},
> * {@code yi} maps to {@code ji}, and {@code id} maps to
> * {@code in}. Since Java SE 17, this is no longer the case.
>
>
> This history was relevant when the property existed. Since this is no longer
> the case, and we're quite a few releases away from 17, can we also remove
> this wording as well. Users on 26 should only be concerned with the "old" to
> "modern" mapping.
Good point. Modified the unnecesarry history.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26419#discussion_r2226240004
