Leverage non-capturing catches

Author: fancyweb

Transport/RoundRobinTransport.php

@@ -51,7 +51,7 @@ public function send(RawMessage $message, Envelope $envelope = null): ?SentMessa
         while ($transport = $this->getNextTransport()) {
             try {
                 return $transport->send($message, $envelope);
-            } catch (TransportExceptionInterface $e) {
+            } catch (TransportExceptionInterface) {
                 $this->deadTransports[$transport] = microtime(true);
             }
         }

Transport/Smtp/EsmtpTransport.php

@@ -114,7 +114,7 @@ private function doEhloCommand(): string
     {
         try {
             $response = $this->executeCommand(sprintf("EHLO %s\r\n", $this->getLocalDomain()), [250]);
-        } catch (TransportExceptionInterface $e) {
+        } catch (TransportExceptionInterface) {
             return parent::executeCommand(sprintf("HELO %s\r\n", $this->getLocalDomain()), [250]);
         }
 
@@ -135,7 +135,7 @@ private function doEhloCommand(): string
             try {
                 $response = $this->executeCommand(sprintf("EHLO %s\r\n", $this->getLocalDomain()), [250]);
                 $this->capabilities = $this->parseCapabilities($response);
-            } catch (TransportExceptionInterface $e) {
+            } catch (TransportExceptionInterface) {
                 return parent::executeCommand(sprintf("HELO %s\r\n", $this->getLocalDomain()), [250]);
             }
         }
@@ -184,7 +184,7 @@ private function handleAuth(array $modes): void
             } catch (TransportExceptionInterface $e) {
                 try {
                     $this->executeCommand("RSET\r\n", [250]);
-                } catch (TransportExceptionInterface $_) {
+                } catch (TransportExceptionInterface) {
                     // ignore this exception as it probably means that the server error was final
                 }
 

Transport/Smtp/SmtpTransport.php

@@ -138,7 +138,7 @@ public function send(RawMessage $message, Envelope $envelope = null): ?SentMessa
             if ($this->started) {
                 try {
                     $this->executeCommand("RSET\r\n", [250]);
-                } catch (TransportExceptionInterface $_) {
+                } catch (TransportExceptionInterface) {
                     // ignore this exception as it probably means that the server error was final
                 }
             }
@@ -276,7 +276,7 @@ public function stop(): void
 
         try {
             $this->executeCommand("QUIT\r\n", [221]);
-        } catch (TransportExceptionInterface $e) {
+        } catch (TransportExceptionInterface) {
         } finally {
             $this->stream->terminate();
             $this->started = false;
@@ -292,7 +292,7 @@ private function ping(): void
 
         try {
             $this->executeCommand("NOOP\r\n", [250]);
-        } catch (TransportExceptionInterface $e) {
+        } catch (TransportExceptionInterface) {
             $this->stop();
         }
     }