On Thu, 23 Jan 2025 22:48:17 GMT, Chen Liang <li...@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
