RFR: 8027370: (tz) Support tzdata2013h

2013-11-05 Thread Aleksej Efimov
Hi, Can I have a review for tzdata2013h integration [1]. The webrev link can be located here [2]. The following test sets were executed on build with fix: test/sun/util/calendar test/java/util/Calendar test/sun/util/resources/TimeZone test/sun/util/calendar test/java/util/TimeZone test/java/

Re: RFR: 8027370: (tz) Support tzdata2013h

2013-11-05 Thread Alan Bateman
On 05/11/2013 16:38, Aleksej Efimov wrote: Hi, Can I have a review for tzdata2013h integration [1]. The webrev link can be located here [2]. The following test sets were executed on build with fix: test/sun/util/calendar test/java/util/Calendar test/sun/util/resources/TimeZone test/sun/util/

Re: RFR: 8027370: (tz) Support tzdata2013h

2013-11-05 Thread Aleksej Efimov
Alan, Thank you for advise. I have executed the same test sets with -XX:-UseMathExactIntrinsics and, as was expected, there is only one failure: sun/util/calendar/zi/TestZoneInfo310.java. -Aleksej On 11/05/2013 09:50 PM, Alan Bateman wrote: On 05/11/2013 16:38, Aleksej Efimov wrote: Hi, Ca

Re: RFR: 8027370: (tz) Support tzdata2013h

2013-11-06 Thread Aleksej Efimov
Hi, We have a fix for JDK-8027848 and it was approved in parallel thread. With '-XX:-UseMathExactIntrinsics' tests run flag and applied fix for JDK-8027848 all tests from the following test sets are passing: test/sun/util/calendar test/java/util/Calendar test/sun/util/resources/TimeZone test/s

Re: RFR: 8027370: (tz) Support tzdata2013h

2013-11-08 Thread Aleksej Efimov
Hi, Can I ask reviewers to look at this fix? As was mentioned, the testing issues described in request were resolved. Thanks in advance, Aleksej On 11/06/2013 11:11 PM, Aleksej Efimov wrote: Hi, We have a fix for JDK-8027848 and it was approved in parallel thread. With '-XX:-UseMathExactInt

Re: RFR: 8027370: (tz) Support tzdata2013h

2013-11-08 Thread Xueming Shen
looks fine. I would assume you've also run the corresponding tests at test/closed repo. -Sherman On 11/5/2013 8:38 AM, Aleksej Efimov wrote: Hi, Can I have a review for tzdata2013h integration [1]. The webrev link can be located here [2]. The following test sets were executed on build with

Re: RFR: 8027370: (tz) Support tzdata2013h

2013-11-15 Thread Seán Coffey
Looks good to me too Aleksej. regards, Sean. On 08/11/2013 16:42, Xueming Shen wrote: looks fine. I would assume you've also run the corresponding tests at test/closed repo. -Sherman On 11/5/2013 8:38 AM, Aleksej Efimov wrote: Hi, Can I have a review for tzdata2013h integration [1]. The we