<i18n dev> RFR: 8059206: (tz) Support tzdata2014i
Aleksej Efimov
aleksej.efimov at oracle.com
Tue Oct 28 14:08:09 UTC 2014
Hi Masayoshi,
Thank you for the review.
Ok, I will change the "Summer Time" to "Daylight Time" before the push.
Best Regards,
Aleksej
On 10/28/2014 04:46 PM, Masayoshi Okutsu wrote:
> Hi Aleksej,
>
> src/java.base/share/classes/sun/util/resources/TimeZoneNames*.java:
> + {"Pacific/Bougainville", new String[] {"Bougainville Standard Time", "BST",
> + "Bougainville Summer Time", "BST",
> + "Bougainville Time", "BT"}},
>
> I'd use "Daylight Time" instead of "Summer Time" because of the recent
> move from Summer to Daylight. No daylight saving time is observed in
> Bougainville, though.
>
> Otherwise, all the changes look good to me.
>
> Thanks,
> Masayoshi
>
> On 10/28/2014 8:57 PM, Aleksej Efimov wrote:
>> Hi,
>>
>> Can I ask for review of new tzdata - 2014i integration fix to JDK9.
>> One change in this release is that new time zone was added -
>> "Pacific/Bougainville". The localized names still needs to be
>> translated - the appropriate comment was added to JDK-8057004.
>>
>> Testing results:
>> JPRT tests: jdk_other jdk_util jdk_text jdk_time - no failures
>> JTREG tests: 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
>> test/closed/java/text/Format/DateFormat - no failures
>>
>> Thank you,
>> Aleksej
>>
>> [1] Bug: https://bugs.openjdk.java.net/browse/JDK-8059206
>> [2] Webrev: http://cr.openjdk.java.net/~aefimov/8059206/9/webrev.00
>
More information about the i18n-dev
mailing list