Add missing return types to interfaces

Author: derrabus

Debug/TraceableEventDispatcher.php

@@ -67,6 +67,9 @@ public function addSubscriber(EventSubscriberInterface $subscriber)
         $this->dispatcher->addSubscriber($subscriber);
     }
 
+    /**
+     * @return void
+     */
     public function removeListener(string $eventName, callable|array $listener)
     {
         if (isset($this->wrappedListeners[$eventName])) {
@@ -79,12 +82,15 @@ public function removeListener(string $eventName, callable|array $listener)
             }
         }
 
-        return $this->dispatcher->removeListener($eventName, $listener);
+        $this->dispatcher->removeListener($eventName, $listener);
     }
 
+    /**
+     * @return void
+     */
     public function removeSubscriber(EventSubscriberInterface $subscriber)
     {
-        return $this->dispatcher->removeSubscriber($subscriber);
+        $this->dispatcher->removeSubscriber($subscriber);
     }
 
     public function getListeners(string $eventName = null): array

EventDispatcherInterface.php

@@ -27,6 +27,8 @@ interface EventDispatcherInterface extends ContractsEventDispatcherInterface
      *
      * @param int $priority The higher this value, the earlier an event
      *                      listener will be triggered in the chain (defaults to 0)
+     *
+     * @return void
      */
     public function addListener(string $eventName, callable $listener, int $priority = 0);
 
@@ -35,14 +37,21 @@ public function addListener(string $eventName, callable $listener, int $priority
      *
      * The subscriber is asked for all the events it is
      * interested in and added as a listener for these events.
+     *
+     * @return void
      */
     public function addSubscriber(EventSubscriberInterface $subscriber);
 
     /**
      * Removes an event listener from the specified events.
+     *
+     * @return void
      */
     public function removeListener(string $eventName, callable $listener);
 
+    /**
+     * @return void
+     */
     public function removeSubscriber(EventSubscriberInterface $subscriber);
 
     /**

ImmutableEventDispatcher.php

@@ -30,21 +30,33 @@ public function dispatch(object $event, string $eventName = null): object
         return $this->dispatcher->dispatch($event, $eventName);
     }
 
+    /**
+     * @return never
+     */
     public function addListener(string $eventName, callable|array $listener, int $priority = 0)
     {
         throw new \BadMethodCallException('Unmodifiable event dispatchers must not be modified.');
     }
 
+    /**
+     * @return never
+     */
     public function addSubscriber(EventSubscriberInterface $subscriber)
     {
         throw new \BadMethodCallException('Unmodifiable event dispatchers must not be modified.');
     }
 
+    /**
+     * @return never
+     */
     public function removeListener(string $eventName, callable|array $listener)
     {
         throw new \BadMethodCallException('Unmodifiable event dispatchers must not be modified.');
     }
 
+    /**
+     * @return never
+     */
     public function removeSubscriber(EventSubscriberInterface $subscriber)
     {
         throw new \BadMethodCallException('Unmodifiable event dispatchers must not be modified.');