<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