Looks fine to me also Sherman. On a side note, we should look at
migrating some closed TZ tests to open repo. I'm not sure why that
wasn't done from first day.
regards,
Sean.
On 09/08/2013 08:20, Yuka Kamiya wrote:
Hi Sherman,
The fix looks good to me.
Thanks,
--
Yuka
(2013/08/09 14:54), Xueming Shen wrote:
Hi,
Please help review the proposed change to update the tz data
in JDK8 from 2013c to 2013d.
http://cr.openjdk.java.net/~sherman/8020054/webrev
http://cr.openjdk.java.net/~sherman/8020054/closed
Tests list below have been run and passed
(except java/time/tck/java/time/chrono/TCKChronology.java with error
expected [Hijrah-umalqura] but found [Islamic Umm al-Qura Calendar],
which
I would assume has nothing to do with tz change)
java/util/TimeZone
java/util/Calendar
java/util/Formatter
java/time
closed/java/util/TimeZone
closed/java/util/Calendar
Thanks!
-Sherman
