<i18n dev> RFR: 8340073: Support "%z" time zone abbreviation format in TZ files [v2]

[Naoto Sato]

On Sat, 14 Sep 2024 09:20:25 GMT, Justin Lu <jlu at openjdk.org> 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