Leverage non-capturing catches

Author: fancyweb

Application.php

@@ -228,7 +228,7 @@ public function doRun(InputInterface $input, OutputInterface $output)
         try {
             // Makes ArgvInput::getFirstArgument() able to distinguish an option from an argument.
             $input->bind($this->getDefinition());
-        } catch (ExceptionInterface $e) {
+        } catch (ExceptionInterface) {
             // Errors must be ignored, full binding/validation happens later when the command is known.
         }
 
@@ -993,7 +993,7 @@ protected function doRunCommand(Command $command, InputInterface $input, OutputI
         try {
             $command->mergeApplicationDefinition();
             $input->bind($command->getDefinition());
-        } catch (ExceptionInterface $e) {
+        } catch (ExceptionInterface) {
             // ignore invalid options/arguments for now, to allow the event listeners to customize the InputDefinition
         }
 

Command/CompleteCommand.php

@@ -184,7 +184,7 @@ private function createCompletionInput(InputInterface $input): CompletionInput
 
         try {
             $completionInput->bind($this->getApplication()->getDefinition());
-        } catch (ExceptionInterface $e) {
+        } catch (ExceptionInterface) {
         }
 
         return $completionInput;
@@ -199,7 +199,7 @@ private function findCommand(CompletionInput $completionInput, OutputInterface $
             }
 
             return $this->getApplication()->find($inputName);
-        } catch (CommandNotFoundException $e) {
+        } catch (CommandNotFoundException) {
         }
 
         return null;

Completion/CompletionInput.php

@@ -183,7 +183,7 @@ protected function parseToken(string $token, bool $parseOptions): bool
     {
         try {
             return parent::parseToken($token, $parseOptions);
-        } catch (RuntimeException $e) {
+        } catch (RuntimeException) {
             // suppress errors, completed input is almost never valid
         }