Default $value to null in Suspension::resume (#19)

Often there's no need for a value, and we just want to resume at the suspension point.

Fixes #11.

Author: kelunik

examples/http-client-suspension.php

@@ -30,7 +30,7 @@ function fetch(string $url): string
     \stream_set_blocking($stream, false);
 
     // wait for connection success/error
-    $watcher = EventLoop::onWritable($stream, fn () => $suspension->resume(null));
+    $watcher = EventLoop::onWritable($stream, fn () => $suspension->resume());
     $suspension->suspend();
     EventLoop::cancel($watcher);
 
@@ -40,7 +40,7 @@ function fetch(string $url): string
     $buffer = '';
 
     // wait for HTTP response
-    $watcher = EventLoop::onReadable($stream, fn () => $suspension->resume(null));
+    $watcher = EventLoop::onReadable($stream, fn () => $suspension->resume());
 
     do {
         $suspension->suspend();

examples/invalid-callback-return.php

@@ -14,7 +14,7 @@
 
     print "Caught SIGINT, exiting..." . PHP_EOL;
 
-    $suspension->resume(null);
+    $suspension->resume();
 
     return new \stdClass();
 });

examples/stdin-timeout.php

@@ -21,10 +21,10 @@
 
     print "Read " . \strlen($chunk) . " bytes" . PHP_EOL;
 
-    $suspension->resume(null);
+    $suspension->resume();
 });
 
-$timeoutWatcher = EventLoop::delay(5000, fn () => $suspension->resume(null));
+$timeoutWatcher = EventLoop::delay(5000, fn () => $suspension->resume());
 
 $suspension->suspend();
 

examples/stdin.php

@@ -21,7 +21,7 @@
 
     print "Read " . \strlen($chunk) . " bytes" . PHP_EOL;
 
-    $suspension->resume(null);
+    $suspension->resume();
 });
 
 $suspension->suspend();

examples/timers.php

@@ -15,7 +15,7 @@
     print "++ Executing watcher created by Loop::delay()" . PHP_EOL;
 
     EventLoop::cancel($repeatWatcher);
-    $suspension->resume(null);
+    $suspension->resume();
 
     print "++ Executed after script ended" . PHP_EOL;
 });

src/EventLoop/Suspension.php

@@ -59,7 +59,7 @@ public function throw(\Throwable $throwable): void
         }
     }
 
-    public function resume(mixed $value): void
+    public function resume(mixed $value = null): void
     {
         if (!$this->pending) {
             throw $this->error ?? new \Error('Must call suspend() before calling resume()');

test/EventLoopTest.php

@@ -38,7 +38,7 @@ public function testOnReadable(): void
             EventLoop::cancel($callbackId);
             $count++;
 
-            $suspension->resume(null);
+            $suspension->resume();
         });
 
         $suspension->suspend();
@@ -56,7 +56,7 @@ public function testOnWritable(): void
             EventLoop::cancel($callbackId);
             $count++;
 
-            $suspension->resume(null);
+            $suspension->resume();
         });
 
         $suspension->suspend();