]> git.vpit.fr Git - perl/modules/Variable-Magic.git/commitdiff
Simplify the timeout logic in t/09-load-threads.t
authorVincent Pit <vince@profvince.com>
Tue, 21 Jul 2015 15:09:57 +0000 (12:09 -0300)
committerVincent Pit <vince@profvince.com>
Tue, 21 Jul 2015 15:16:32 +0000 (12:16 -0300)
The old loop could actually never be executed more than one iteration.
A real good implementation would check whether any of the peers has exited,
but it would involve much more work.

t/09-load-threads.t

index 65a5d2d09a7d728449a44d80f3c222bdb5139b4f..60d6acf1499c30757e77ad51ae3b4cd7fca6f4f3 100644 (file)
@@ -178,14 +178,12 @@ sub sync_master {
 
  LOCK: {
   lock $locks_up[$id];
-  for (1 .. 100) {
-   my $timeout = time() + 2;
-   until ($locks_up[$id] == $peers) {
-    if (cond_timedwait $locks_up[$id], $timeout) {
-     last LOCK;
-    } else {
-     return 0;
-    }
+  my $timeout = time() + 10;
+  until ($locks_up[$id] == $peers) {
+   if (cond_timedwait $locks_up[$id], $timeout) {
+    last LOCK;
+   } else {
+    return 0;
    }
   }
  }