<i18n dev> RFR: 8340477: Remove JDK1.1 compatible behavior for "EST", "MST", and "HST" time zones [v2]
Naoto Sato
naoto at openjdk.org
Wed Oct 16 19:39:44 UTC 2024
On Wed, 16 Oct 2024 18:33:25 GMT, Justin Lu <jlu at openjdk.org> wrote:
>> Naoto Sato has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Reflects review comments
>
> src/java.base/share/classes/sun/util/calendar/ZoneInfoFile.java line 239:
>
>> 237: case "EST" -> aliases.put("EST", "America/Panama");
>> 238: case "MST" -> aliases.put("MST", "America/Phoenix");
>> 239: default -> aliases.put(key, ZoneId.SHORT_IDS.get(key));
>
> "EST" and "MST" case are no longer needed? ZoneId should already map them to "America/Panama" and "America/Phoenix". If we no longer offer the old mapping, then we can safely follow ZoneId.SHORT_IDS.
Right. Removed them.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21545#discussion_r1803689515
More information about the i18n-dev
mailing list