moved timersgc foreach loop into callback

cameronjacobson · cameronjacobson · commit d39e57d4677b · 2012-12-22T16:10:44.000-05:00
diff --git a/src/React/EventLoop/LibEventLoop.php b/src/React/EventLoop/LibEventLoop.php
@@ -161,10 +161,6 @@ protected function addTimerInternal($interval, $callback, $periodic = false)
             throw new \InvalidArgumentException('The callback must be a callable object.');
         }
 
-        foreach ($this->timersGc as $resource) {
-            event_free($resource);
-        }
-
         $timer = (object) array(
             'loop' => $this,
             'resource' => $resource = event_new(),
@@ -177,6 +173,9 @@ protected function addTimerInternal($interval, $callback, $periodic = false)
         $timer->signature = spl_object_hash($timer);
 
         $callback = function () use ($timer) {
+            foreach ($this->timersGc as $resource) {
+                event_free($resource);
+            }
             if ($timer->cancelled === false) {
                 call_user_func($timer->callback, $timer->signature, $timer->loop);
 
