On Wed, 16 Oct 2024 17:46:20 GMT, Naoto Sato <na...@openjdk.org> wrote:
> Removing the old compatibility behavior switch for JDK1.1. With the
> integration of tzdata2024b, definitions of those three (deprecated) IDs
> changed again, so maintaining the JDK11 compatibility got less useful.
> Removing the compatibility behavior makes the code and test simpler. A CSR
> for the behavior change is drafted too.
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". We no longer offer the old mapping, so
we can safely follow ZoneId.SHORT_IDS.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21545#discussion_r1803613511
