On Tue, 7 Apr 2026 20:02:07 GMT, Naoto Sato <[email protected]> wrote:

>> This PR adds support for the std/dstOffset attributes in CLDR metazone 
>> definitions. These attributes were introduced in CLDR v48 to address 
>> differences between the IANA TZDB vanguard and rearguard formats. At the 
>> time JDK adopted CLDR v48, this support was not implemented because the JDK 
>> handles these differences separately (see JDK-8212970).
>> 
>> More recently, the America/Vancouver time zone is planning to adopt a 
>> year-round fixed offset. To avoid mismatches between standard and daylight 
>> saving time naming, CLDR has started using these attributes for that zone. 
>> This change prepares the JDK for upcoming TZDB updates and the corresponding 
>> CLDR changes.
>
> Naoto Sato has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Code simplification

make/jdk/src/classes/build/tools/cldrconverter/CLDRConverter.java line 144:

> 142: 
> 143:     // Map of explicit dst offsets for metazones
> 144:     // key: metazone name

Small suggestion on this comment. I think the key is the time zone ID (for 
example, "Europe/Dublin"), not the metazone name (for example, "Irish").

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30594#discussion_r3047664537

Reply via email to