Hi,

Please, review the tzdata2014f integration (with tzdata2014e related changes included too) [1] fix to JDK9: http://cr.openjdk.java.net/~aefimov/8049343/9/webrev.01/

The tzdata2014f changes are extensive and relates mostly to timezone short names changes + "Asia/Srednekolymsk" time zone were added. Almost complete list of changes can be found in the JBS bug description [1], plus some changes wasn't documented in tzdata release notes - for such cases raw tzdata diff was used for the names modifications.

Two issues with JSR310 implementation were discovered during integration process: First issue is related to the internal representation of the '24:00' value. The JSR310 implementation treats this value as a next day 00:00 time. The workaround already exists in JSR310 code for similar entries and this failure is resolved in similar way [2] as part of this update. For the second issue JDK-8051641 [3] was filled and 'sun/util/calendar/zi/TestZoneInfo310.java' test is the only one that fails with this tzdata.
Other time zone related tests [4] passes without failures.

Thank you,
Aleksej

[1] https://bugs.openjdk.java.net/browse/JDK-8049343
[2] http://cr.openjdk.java.net/~aefimov/8049343/9/webrev.01/src/java.base/share/classes/sun/util/calendar/ZoneInfoFile.java.patch
[3] https://bugs.openjdk.java.net/browse/JDK-8051641
[4] TZ related test sets: test/sun/util/calendar test/java/util/Calendar test/sun/util/resources/TimeZone test/sun/util/calendar test/java/util/TimeZone test/java/time\ test/java/util/Formatter test/closed/java/util/Calendar test/closed/java/util/TimeZone

Reply via email to