On Sat, 14 Sep 2024 09:20:25 GMT, Justin Lu <[email protected]> wrote:
>> Naoto Sato has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Do not add all links
>
> src/java.base/share/classes/sun/util/cldr/CLDRTimeZoneNameProviderImpl.java
> line 270:
>
>> 268: ResourceBundle fd = lr.getJavaTimeFormatData();
>> 269: var zi = ZoneInfoFile.getZoneInfo(id);
>> 270: if (zi == null) {
>
> This is an added fallback, and is not related to the "%z" logic right?
Right. Since `getZoneInfo()` could return `null`, this is just for safety.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20979#discussion_r1761534541
