Run tests on PHP 8.4 and update test environment

Author: PaulRotmann

.github/workflows/ci.yml

@@ -7,10 +7,11 @@ on:
 jobs:
   PHPUnit:
     name: PHPUnit (PHP ${{ matrix.php }})
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-24.04
     strategy:
       matrix:
         php:
+          - 8.4
           - 8.3
           - 8.2
           - 8.1
@@ -38,7 +39,7 @@ jobs:
 
   PHPUnit-hhvm:
     name: PHPUnit (HHVM)
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-24.04
     continue-on-error: true
     steps:
       - uses: actions/checkout@v4

src/Client.php

@@ -28,7 +28,7 @@ class Client
      * @param ?LoopInterface $loop
      * @param ?MulticastFactory $multicast
      */
-    public function __construct(LoopInterface $loop = null, MulticastFactory $multicast = null)
+    public function __construct(?LoopInterface $loop = null, ?MulticastFactory $multicast = null)
     {
         $this->loop = $loop ?: Loop::get();
         $this->multicast = $multicast ?: new MulticastFactory($this->loop);
@@ -46,13 +46,18 @@ public function search($searchTarget = 'ssdp:all', $mx = 2)
         $socket = $this->multicast->createSender();
         // TODO: The TTL for the IP packet SHOULD default to 2 and SHOULD be configurable.
 
-        $timer = $this->loop->addTimer($mx, function() use ($socket, &$deferred) {
+        // Initialize deferred first
+        $deferred = new Deferred();
+
+        // Then use it in the timer
+        $timer = $this->loop->addTimer($mx, function() use ($socket, $deferred) {
             $deferred->resolve();
             $socket->close();
         });
 
+        // Redefine the deferred with the proper canceller now that timer exists
         $loop = $this->loop;
-        $deferred = new Deferred(function () use ($socket, &$timer, $loop) {
+        $deferred = new Deferred(function () use ($socket, $timer, $loop) {
             // canceling resulting promise cancels timer and closes socket
             $loop->cancelTimer($timer);
             $socket->close();