<i18n dev> RFR: 8347955: TimeZone methods to stream the available timezone IDs [v3]

Andrey Turbanov aturbanov at openjdk.org
Fri Jan 24 08:38:46 UTC 2025


On Thu, 23 Jan 2025 22:48:17 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Right, it can be removed. Actually, I think `getAvailableIDs()` and `getAvailableIDs(int)` also do not need to be synchronized. The internal IDs are eagerly initialized in the static block of `ZoneInfoFile.java` and are effectively final and not modified.
>
> Yep; their synchronized are incorrect. The object monitor lock should have been on `ZoneInfoFile.class` to be effective.

Related PR waiting for review - https://github.com/openjdk/jdk/pull/17441

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23251#discussion_r1928323534


More information about the i18n-dev mailing list