On Sat, 3 Oct 2026 10:39:34 GMT, Thomas Zimmermann <[email protected]> wrote:

>> Naoto Sato has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Addresses review comments
>
> src/java.base/share/classes/sun/util/locale/provider/LocaleResources.java 
> line 374:
> 
>> 372:                 offsets.put(entry.substring(0, separator), 
>> ZoneOffset.of(entry.substring(separator + 1)));
>> 373:             }
>> 374:             return offsets;
> 
> Suggestion:
> 
>             return Map.copyOf(offsets);

The map is not intended to be exposed to clients, so there's no need to create 
a defensive copy.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33133#discussion_r4186133854

Reply via email to