On Tue, 10 Sep 2024 22:17:51 GMT, Justin Lu <j...@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/cldrconverter/CLDRConverter.java line 1375:
>
>> 1373: Files.walk(Path.of(tzDataDir), 1, FileVisitOption.FOLLOW_LINKS)
>> 1374: .filter(p -> p.toFile().isFile())
>> 1375: .filter(p ->
>> p.getFileName().toString().matches("africa|antarctica|asia|australasia|backward|backzone|etcetera|europe|factory|northamerica|southamerica"))
>
> I am just curious, why is this needed?
It would otherwise scan all files under the directory, and fails with
`iso3166.tab` or `zone.tab`, which include "LI" as the country code at the
beginning of the line, that interfere with "Link" match.
Having said that, the default list was different from the one in
`TzdbZoneRulesProvider`. I removed extra file names.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20940#discussion_r1752889391
