On Wed, 16 Oct 2024 18:33:25 GMT, Justin Lu <[email protected]> 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
