Hi Aleksej,
I took a look at the localized TimeZoneNames_*.java files. They do not
contain generic time zone names for JSR310...
I think we can file a separate bug to track that issue.
thanks,
-michael
On 13年10月10日 09:54 下午, Masayoshi Okutsu wrote:
Hi Aleksej,
Here are my review comments.
- The copyright header of the data files shouldn't be removed.
- TimeZoneNames.java:
- "Middle Europe Time", "MET"}},
+ "MET", "MET"}},
I don't think the long name should be changed.
I didn't review the localized TimeZoneNames_*.java files. If L10N Team
is OK with them, I'm fine.
Thanks,
Masayoshi
On 10/10/2013 10:30 PM, Aleksej Efimov wrote:
Hi,
Please, review the changes [1] needed to address the tz data update
in JDK 8 from tzdata2013d to tzdata2013g.
The brief list of changes:
1. tzdata2013g data was integrated to tzdb data files
(make/sun/javazic/tzdata/* changes) and to sun/util/calendar test
data files (test/sun/util/calendar/zi/tzdata/* changes).
2. a) Updates to long time zone names
b) Updates to short name changes to address corresponding changes in
tzdata2013e(WIT/CIT/EIT/WARST -> WIB/WITA/WIT/ART)
c) Removed unused ACT[] array
d) Added "Europe/Busingen" time zone name
All this changes a)->d) relates to
src/share/classes/sun/util/resources/TimeZoneNames*.java files
The following tests were executed on JDK 8 with fix:
test/java/util/TimeZone
test/java/util/Calendar
test/java/util/Formatter
test/sun/util/calendar
test/java/time
Testing result: All test passed
Thanks!
Aleksej
[1] http://cr.openjdk.java.net/~aefimov/8025255/8/webrev.00/
<http://cr.openjdk.java.net/%7Eaefimov/8025255/8/webrev.00/>
