On Tue, 28 Jan 2025 17:06:24 GMT, Shaojin Wen <s...@openjdk.org> wrote:

>> ZoneOffset.MINUTES_15_CACHE uses AtomicReferenceArray to replace 
>> ConcurrentMap to avoid object allocation caused by boxing from int to 
>> Integer during access.
>
> Shaojin Wen has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   from @liach

The change looks good to me. Maybe you could write a test case for all the 
quarters that indeed return cached ZoneOffsets? Also, titles (PR/Issue) can 
reflect the renamed cache name

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

PR Comment: https://git.openjdk.org/jdk/pull/23337#issuecomment-2620289635

Reply via email to