Please review two corrections for java.time

David Holmes david.holmes at oracle.com
Mon Sep 9 02:43:52 UTC 2013


Hi Roger,

On 7/09/2013 3:58 AM, roger riggs wrote:
> Please review for two corrections:
>
> -  The java/time/tck/java/time/TCKLocalTime test failed on a slow machine;
>      the test should be more lenient.   The test is not appropriate for
> a conformance test
>      and is moved to java/time/test/java/time/TestLocalTime.

As per the bug report the issue is not slow machines per-se but the use 
of Xcomp when running the test. I don't think the jck should ever be run 
with Xcomp. It will be interesting to see if the change from 100ms to 
500ms cures the problem on all machines.

Thanks,
David

> - The javadoc for the JapaneseEra.MEIJI era should indicate the start
> date is 1868-01-01
>    to be consistent with java.util.Calendar.  Note that java.time does
> not permit dates before Meiji 6
>    to be created since the calendar is not clearly defined until then.
>
> Webrev: http://cr.openjdk.java.net/~rriggs/webrev-localtime-now-8023639/
>
> Thanks, Roger
>
>



More information about the core-libs-dev mailing list