<i18n dev> RFR: 8025255: (tz) Support tzdata2013g

Masayoshi Okutsu masayoshi.okutsu at oracle.com
Thu Oct 10 21:54:07 PDT 2013


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/>
>



More information about the i18n-dev mailing list