<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