<i18n dev> RFR: 8341484: TimeZone.toZoneId() throws exception when using old mapping for "HST"

Naoto Sato naoto at openjdk.org
Tue Oct 8 17:49:02 UTC 2024


On Thu, 3 Oct 2024 19:46:48 GMT, Justin Lu <jlu at openjdk.org> wrote:

> Please review this PR which fixes an unexpected exception for `TimeZone.toZoneId("HST")` when the old mapping system property is used.
> 
> The culprit is in `TimeZone.toZoneId0()`.
> 
> This method provides workarounds for "EST", "MST", and "HST" when the old mapping system property is true. However, it seems that "HST" was added with "America/Honolulu" when it should have been "Pacific/Honolulu".
> 
> Added a basic test.

This is indeed surprising that this has not been revealed by now. I guess nobody used this old mapping before... Anyway, I am now trying to get rid of this old mapping behavior altogether (https://bugs.openjdk.org/browse/JDK-8340477), so the entire code will be gone.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/21332#issuecomment-2400469928


More information about the i18n-dev mailing list