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
