changeset 8181:35bb1c7f227c

8023556: Update javadoc for start of Meiji era Summary: correct the javadoc in JapaneseEra.MEIJI to match the implementation Reviewed-by: darcy, sherman
author rriggs
date Sat, 14 Sep 2013 13:55:06 -0400
parents 3255a4e348a1
children a88254c70ad6 ff6c76f7733e
files src/share/classes/java/time/chrono/JapaneseEra.java test/java/time/test/java/time/TestLocalTime.java
diffstat 2 files changed, 20 insertions(+), 9 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/classes/java/time/chrono/JapaneseEra.java	Sat Sep 14 13:55:04 2013 -0400
+++ b/src/share/classes/java/time/chrono/JapaneseEra.java	Sat Sep 14 13:55:06 2013 -0400
@@ -104,7 +104,7 @@
     static final sun.util.calendar.Era[] ERA_CONFIG;
 
     /**
-     * The singleton instance for the 'Meiji' era (1868-09-08 - 1912-07-29)
+     * The singleton instance for the 'Meiji' era (1868-01-01 - 1912-07-29)
      * which has the value -1.
      */
     public static final JapaneseEra MEIJI = new JapaneseEra(-1, LocalDate.of(1868, 1, 1));
--- a/test/java/time/test/java/time/TestLocalTime.java	Sat Sep 14 13:55:04 2013 -0400
+++ b/test/java/time/test/java/time/TestLocalTime.java	Sat Sep 14 13:55:06 2013 -0400
@@ -73,6 +73,9 @@
  */
 @Test
 public class TestLocalTime extends AbstractTest {
+    static final long NANOS_PER_SECOND = 1_000_000_000L;
+    static final long NANOS_PER_MINUTE = 60 * NANOS_PER_SECOND;
+    static final long NANOS_PER_DAY = 24 * 60 * NANOS_PER_MINUTE;
 
     //-----------------------------------------------------------------------
     @Test
@@ -182,19 +185,27 @@
     // now()
     //-----------------------------------------------------------------------
     @Test
+    @SuppressWarnings("unused")
     public void now() {
         // Warmup the TimeZone data so the following test does not include
         // one-time initialization
-        LocalTime expected = LocalTime.now(Clock.systemDefaultZone());
+        LocalTime.now(Clock.systemDefaultZone());
 
-        expected = LocalTime.now(Clock.systemDefaultZone());
-        LocalTime test = LocalTime.now();
-        long diff = test.toNanoOfDay() - expected.toNanoOfDay();
-        if (diff < 0) {
-            // Adjust if for rollover around midnight
-            diff +=  24 * 60 * 60 * 1_000_000_000L; // Nanos Per Day
+        long diff = Integer.MAX_VALUE;
+        for (int i = 0; i < 2; i++) {
+            LocalTime expected = LocalTime.now(Clock.systemDefaultZone());
+            LocalTime test = LocalTime.now();
+            diff = test.toNanoOfDay() - expected.toNanoOfDay();
+            // Normalize for wrap-around midnight
+            diff = Math.floorMod(NANOS_PER_DAY + diff, NANOS_PER_DAY);
+            if (diff < 100000000) {
+                break;
+            }
+            // A second iteration may be needed if the clock changed
+            // due to a DST change between the two calls to now.
         }
-        assertTrue(diff < 500000000);  // less than 0.5 secs
+        assertTrue(diff < 100000000,   // less than 0.1 sec
+                "LocalTime.now  vs LocalTime.now(Clock.systemDefaultZone()) not close");
     }
 
 }