RFR: 8339803: Acknowledge case insensitive unambiguous keywords in tzdata files [v2]

Naoto Sato naoto at openjdk.org
Mon Sep 16 19:33:09 UTC 2024


On Mon, 16 Sep 2024 18:57:42 GMT, Stephen Colebourne <scolebourne at openjdk.org> wrote:

>> Naoto Sato has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   tz files aligned with the default TzdbZoneRulesProvider list
>
> make/jdk/src/classes/build/tools/tzdb/TzdbZoneRulesProvider.java line 308:
> 
>> 306:             if (off < tokens.length) {
>> 307:                 String dayRule = tokens[off++];
>> 308:                 if (dayRule.regionMatches(true, 0, "last", 0, 4)) {
> 
> This isn't correct, as per the mailing list:
> 
>> > can "last" contain uppercase letters, or
>> > does it have to be exactly "last"?
>> 
>> In that case the name is "lastSunday", and it can be abbreviated
>> "lastSu" or "Lastsu" or "LASTSUNDA" or whatever.

Why is it not? IIUC, abbreviation should be unambiguous, so "last" part is always case-insensitive 4 character length.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20940#discussion_r1761768765


More information about the build-dev mailing list