Author: peter_firmstone
Date: Mon Dec 23 23:27:41 2013
New Revision: 1553229

URL: http://svn.apache.org/r1553229
Log:
Fix waiting too long after spurious wakeup.

Modified:
    
river/jtsk/skunk/qa_refactor/trunk/qa/src/com/sun/jini/test/impl/norm/LeaseExpirationTest.java

Modified: 
river/jtsk/skunk/qa_refactor/trunk/qa/src/com/sun/jini/test/impl/norm/LeaseExpirationTest.java
URL: 
http://svn.apache.org/viewvc/river/jtsk/skunk/qa_refactor/trunk/qa/src/com/sun/jini/test/impl/norm/LeaseExpirationTest.java?rev=1553229&r1=1553228&r2=1553229&view=diff
==============================================================================
--- 
river/jtsk/skunk/qa_refactor/trunk/qa/src/com/sun/jini/test/impl/norm/LeaseExpirationTest.java
 (original)
+++ 
river/jtsk/skunk/qa_refactor/trunk/qa/src/com/sun/jini/test/impl/norm/LeaseExpirationTest.java
 Mon Dec 23 23:27:41 2013
@@ -360,13 +360,15 @@ public class LeaseExpirationTest extends
            if (count != 0) 
                return false;
 
-           long now = System.currentTimeMillis();
+           long duration = setExpiration - System.currentTimeMillis();
+            if (duration > 0) logger.log(Level.INFO, "Waiting for " + 
(duration) 
+                    + " ms for set lease to expire");
+                
            synchronized (this) {
-               while (setExpiration > now) {
-                   logger.log(Level.INFO, "Waiting for " + (setExpiration - 
now) 
-                        + " ms for set lease to expire");
-                   wait(setExpiration - now);
-                    now = System.currentTimeMillis();
+                duration = setExpiration - System.currentTimeMillis();
+               while (duration > 0) {
+                   wait(duration);
+                    duration = setExpiration - System.currentTimeMillis();
                 }
            }
            return true;


Reply via email to