<i18n dev> RFR: 8294357: (tz) Update Timezone Data to 2022d
Naoto Sato
naoto at openjdk.org
Fri Sep 30 00:36:54 UTC 2022
On Wed, 28 Sep 2022 03:40:06 GMT, Yoshiki Sato <ysatowse at openjdk.org> wrote:
> Please review this PR. The change include some code changes in ZoneInfoFile.java and TestZoneInfo310.java since tzdata2022d breaks TestZoneInfo310.java.
Some minor comments
src/java.base/share/classes/sun/util/calendar/ZoneInfoFile.java line 588:
> 586: // will not work for Asia/Gaza and Asia/Hebron which follow
> 587: // Palestine DST rules.
> 588: // No need of hacking for Plaestine DST rules from tz2022d
Should this comment be removed, or at least modified to be better explained?
test/jdk/sun/util/calendar/zi/TestZoneInfo310.java line 201:
> 199: zid.equals("Asia/Tehran") || // last rule mismatch
> 200: zid.equals("Asia/Gaza") ||
> 201: zid.equals("Asia/Hebron") ||
It'd be helpful to append `// uses "Palestine" rule`.
-------------
PR: https://git.openjdk.org/jdk/pull/10460
More information about the i18n-dev
mailing list