CS fixes

nicolas-grekas · nicolas-grekas · commit 8eb6dc555bfb · 2025-07-10T10:14:14.000+02:00
diff --git a/Exception/ProcessFailedException.php b/Exception/ProcessFailedException.php
@@ -28,15 +28,15 @@ public function __construct(Process $process)
             throw new InvalidArgumentException('Expected a failed process, but the given process was successful.');
         }
 
-        $error = sprintf('The command "%s" failed.'."\n\nExit Code: %s(%s)\n\nWorking directory: %s",
+        $error = \sprintf('The command "%s" failed.'."\n\nExit Code: %s(%s)\n\nWorking directory: %s",
             $process->getCommandLine(),
             $process->getExitCode(),
             $process->getExitCodeText(),
             $process->getWorkingDirectory()
         );
 
         if (!$process->isOutputDisabled()) {
-            $error .= sprintf("\n\nOutput:\n================\n%s\n\nError Output:\n================\n%s",
+            $error .= \sprintf("\n\nOutput:\n================\n%s\n\nError Output:\n================\n%s",
                 $process->getOutput(),
                 $process->getErrorOutput()
             );
diff --git a/Exception/ProcessSignaledException.php b/Exception/ProcessSignaledException.php
@@ -26,7 +26,7 @@ public function __construct(Process $process)
     {
         $this->process = $process;
 
-        parent::__construct(sprintf('The process has been signaled with signal "%s".', $process->getTermSignal()));
+        parent::__construct(\sprintf('The process has been signaled with signal "%s".', $process->getTermSignal()));
     }
 
     public function getProcess(): Process
diff --git a/Exception/ProcessTimedOutException.php b/Exception/ProcessTimedOutException.php
@@ -31,7 +31,7 @@ public function __construct(Process $process, int $timeoutType)
         $this->process = $process;
         $this->timeoutType = $timeoutType;
 
-        parent::__construct(sprintf(
+        parent::__construct(\sprintf(
             'The process "%s" exceeded the timeout of %s seconds.',
             $process->getCommandLine(),
             $this->getExceededTimeout()
@@ -67,7 +67,7 @@ public function getExceededTimeout(): ?float
         return match ($this->timeoutType) {
             self::TYPE_GENERAL => $this->process->getTimeout(),
             self::TYPE_IDLE => $this->process->getIdleTimeout(),
-            default => throw new \LogicException(sprintf('Unknown timeout type "%d".', $this->timeoutType)),
+            default => throw new \LogicException(\sprintf('Unknown timeout type "%d".', $this->timeoutType)),
         };
     }
 }
diff --git a/ExecutableFinder.php b/ExecutableFinder.php
@@ -74,7 +74,7 @@ public function find(string $name, ?string $default = null, array $extraDirs = [
             $suffixes = $this->suffixes;
             $suffixes = array_merge($suffixes, $pathExt ? explode(\PATH_SEPARATOR, $pathExt) : ['.exe', '.bat', '.cmd', '.com']);
         }
-        $suffixes = '' !== pathinfo($name, PATHINFO_EXTENSION) ? array_merge([''], $suffixes) : array_merge($suffixes, ['']);
+        $suffixes = '' !== pathinfo($name, \PATHINFO_EXTENSION) ? array_merge([''], $suffixes) : array_merge($suffixes, ['']);
         foreach ($suffixes as $suffix) {
             foreach ($dirs as $dir) {
                 if ('' === $dir) {
diff --git a/InputStream.php b/InputStream.php
@@ -50,7 +50,7 @@ public function write(mixed $input)
             return;
         }
         if ($this->isClosed()) {
-            throw new RuntimeException(sprintf('"%s" is closed.', static::class));
+            throw new RuntimeException(\sprintf('"%s" is closed.', static::class));
         }
         $this->input[] = ProcessUtils::validateInput(__METHOD__, $input);
     }
diff --git a/PhpProcess.php b/PhpProcess.php
@@ -52,7 +52,7 @@ public function __construct(string $script, ?string $cwd = null, ?array $env = n
 
     public static function fromShellCommandline(string $command, ?string $cwd = null, ?array $env = null, mixed $input = null, ?float $timeout = 60): static
     {
-        throw new LogicException(sprintf('The "%s()" method cannot be called when using "%s".', __METHOD__, self::class));
+        throw new LogicException(\sprintf('The "%s()" method cannot be called when using "%s".', __METHOD__, self::class));
     }
 
     /**
diff --git a/PhpSubprocess.php b/PhpSubprocess.php
@@ -75,7 +75,7 @@ public function __construct(array $command, ?string $cwd = null, ?array $env = n
 
     public static function fromShellCommandline(string $command, ?string $cwd = null, ?array $env = null, mixed $input = null, ?float $timeout = 60): static
     {
-        throw new LogicException(sprintf('The "%s()" method cannot be called when using "%s".', __METHOD__, self::class));
+        throw new LogicException(\sprintf('The "%s()" method cannot be called when using "%s".', __METHOD__, self::class));
     }
 
     public function start(?callable $callback = null, array $env = []): void
diff --git a/Pipes/AbstractPipes.php b/Pipes/AbstractPipes.php
@@ -101,7 +101,7 @@ protected function write(): ?array
             } elseif (!isset($this->inputBuffer[0])) {
                 if (!\is_string($input)) {
                     if (!\is_scalar($input)) {
-                        throw new InvalidArgumentException(sprintf('"%s" yielded a value of type "%s", but only scalars and stream resources are supported.', get_debug_type($this->input), get_debug_type($input)));
+                        throw new InvalidArgumentException(\sprintf('"%s" yielded a value of type "%s", but only scalars and stream resources are supported.', get_debug_type($this->input), get_debug_type($input)));
                     }
                     $input = (string) $input;
                 }
diff --git a/Pipes/WindowsPipes.php b/Pipes/WindowsPipes.php
@@ -53,7 +53,7 @@ public function __construct(mixed $input, bool $haveReadSupport)
             set_error_handler(function ($type, $msg) use (&$lastError) { $lastError = $msg; });
             for ($i = 0;; ++$i) {
                 foreach ($pipes as $pipe => $name) {
-                    $file = sprintf('%s\\sf_proc_%02X.%s', $tmpDir, $i, $name);
+                    $file = \sprintf('%s\\sf_proc_%02X.%s', $tmpDir, $i, $name);
 
                     if (!$h = fopen($file.'.lock', 'w')) {
                         if (file_exists($file.'.lock')) {
diff --git a/Process.php b/Process.php
@@ -340,7 +340,7 @@ public function start(?callable $callback = null, array $env = [])
         }
 
         if (!is_dir($this->cwd)) {
-            throw new RuntimeException(sprintf('The provided cwd "%s" does not exist.', $this->cwd));
+            throw new RuntimeException(\sprintf('The provided cwd "%s" does not exist.', $this->cwd));
         }
 
         $process = @proc_open($commandline, $descriptors, $this->processPipes->pipes, $this->cwd, $envPairs, $this->options);
@@ -1197,7 +1197,7 @@ public function setOptions(array $options)
         foreach ($options as $key => $value) {
             if (!\in_array($key, $existingOptions)) {
                 $this->options = $defaultOptions;
-                throw new LogicException(sprintf('Invalid option "%s" passed to "%s()". Supported options are "%s".', $key, __METHOD__, implode('", "', $existingOptions)));
+                throw new LogicException(\sprintf('Invalid option "%s" passed to "%s()". Supported options are "%s".', $key, __METHOD__, implode('", "', $existingOptions)));
             }
             $this->options[$key] = $value;
         }
@@ -1453,10 +1453,10 @@ private function doSignal(int $signal, bool $throwException): bool
         }
 
         if ('\\' === \DIRECTORY_SEPARATOR) {
-            exec(sprintf('taskkill /F /T /PID %d 2>&1', $pid), $output, $exitCode);
+            exec(\sprintf('taskkill /F /T /PID %d 2>&1', $pid), $output, $exitCode);
             if ($exitCode && $this->isRunning()) {
                 if ($throwException) {
-                    throw new RuntimeException(sprintf('Unable to kill the process (%s).', implode(' ', $output)));
+                    throw new RuntimeException(\sprintf('Unable to kill the process (%s).', implode(' ', $output)));
                 }
 
                 return false;
@@ -1466,12 +1466,12 @@ private function doSignal(int $signal, bool $throwException): bool
                 $ok = @proc_terminate($this->process, $signal);
             } elseif (\function_exists('posix_kill')) {
                 $ok = @posix_kill($pid, $signal);
-            } elseif ($ok = proc_open(sprintf('kill -%d %d', $signal, $pid), [2 => ['pipe', 'w']], $pipes)) {
+            } elseif ($ok = proc_open(\sprintf('kill -%d %d', $signal, $pid), [2 => ['pipe', 'w']], $pipes)) {
                 $ok = false === fgets($pipes[2]);
             }
             if (!$ok) {
                 if ($throwException) {
-                    throw new RuntimeException(sprintf('Error while sending signal "%s".', $signal));
+                    throw new RuntimeException(\sprintf('Error while sending signal "%s".', $signal));
                 }
 
                 return false;
@@ -1528,7 +1528,7 @@ function ($m) use (&$env, $uid) {
 
         if (!$comSpec && $comSpec = (new ExecutableFinder())->find('cmd.exe')) {
             // Escape according to CommandLineToArgvW rules
-            $comSpec = '"'.preg_replace('{(\\\\*+)"}', '$1$1\"', $comSpec) .'"';
+            $comSpec = '"'.preg_replace('{(\\\\*+)"}', '$1$1\"', $comSpec).'"';
         }
 
         $cmd = ($comSpec ?? 'cmd').' /V:ON /E:ON /D /C ('.str_replace("\n", ' ', $cmd).')';
@@ -1547,7 +1547,7 @@ function ($m) use (&$env, $uid) {
     private function requireProcessIsStarted(string $functionName): void
     {
         if (!$this->isStarted()) {
-            throw new LogicException(sprintf('Process must be started before calling "%s()".', $functionName));
+            throw new LogicException(\sprintf('Process must be started before calling "%s()".', $functionName));
         }
     }
 
@@ -1559,7 +1559,7 @@ private function requireProcessIsStarted(string $functionName): void
     private function requireProcessIsTerminated(string $functionName): void
     {
         if (!$this->isTerminated()) {
-            throw new LogicException(sprintf('Process must be terminated before calling "%s()".', $functionName));
+            throw new LogicException(\sprintf('Process must be terminated before calling "%s()".', $functionName));
         }
     }
 
@@ -1589,7 +1589,7 @@ private function replacePlaceholders(string $commandline, array $env): string
     {
         return preg_replace_callback('/"\$\{:([_a-zA-Z]++[_a-zA-Z0-9]*+)\}"/', function ($matches) use ($commandline, $env) {
             if (!isset($env[$matches[1]]) || false === $env[$matches[1]]) {
-                throw new InvalidArgumentException(sprintf('Command line is missing a value for parameter "%s": ', $matches[1]).$commandline);
+                throw new InvalidArgumentException(\sprintf('Command line is missing a value for parameter "%s": ', $matches[1]).$commandline);
             }
 
             return $this->escapeArgument($env[$matches[1]]);
diff --git a/ProcessUtils.php b/ProcessUtils.php
@@ -56,7 +56,7 @@ public static function validateInput(string $caller, mixed $input): mixed
                 return new \IteratorIterator($input);
             }
 
-            throw new InvalidArgumentException(sprintf('"%s" only accepts strings, Traversable objects or stream resources.', $caller));
+            throw new InvalidArgumentException(\sprintf('"%s" only accepts strings, Traversable objects or stream resources.', $caller));
         }
 
         return $input;
diff --git a/Tests/ExecutableFinderTest.php b/Tests/ExecutableFinderTest.php
@@ -149,7 +149,7 @@ public function testFindBatchExecutableOnWindows()
      */
     public function testEmptyDirInPath()
     {
-        putenv(sprintf('PATH=%s%s', \dirname(\PHP_BINARY), \PATH_SEPARATOR));
+        putenv(\sprintf('PATH=%s%s', \dirname(\PHP_BINARY), \PATH_SEPARATOR));
 
         try {
             touch('executable');
@@ -158,7 +158,7 @@ public function testEmptyDirInPath()
             $finder = new ExecutableFinder();
             $result = $finder->find('executable');
 
-            $this->assertSame(sprintf('.%sexecutable', \DIRECTORY_SEPARATOR), $result);
+            $this->assertSame(\sprintf('.%sexecutable', \DIRECTORY_SEPARATOR), $result);
         } finally {
             unlink('executable');
         }
diff --git a/Tests/ProcessTest.php b/Tests/ProcessTest.php
@@ -168,7 +168,7 @@ public function testAllOutputIsActuallyReadOnTermination()
         // another byte which will never be read.
         $expectedOutputSize = PipesInterface::CHUNK_SIZE * 2 + 2;
 
-        $code = sprintf('echo str_repeat(\'*\', %d);', $expectedOutputSize);
+        $code = \sprintf('echo str_repeat(\'*\', %d);', $expectedOutputSize);
         $p = $this->getProcessForCode($code);
 
         $p->start();
@@ -359,7 +359,7 @@ public static function chainedCommandsOutputProvider()
      */
     public function testChainedCommandsOutput($expected, $operator, $input)
     {
-        $process = $this->getProcess(sprintf('echo %s %s echo %s', $input, $operator, $input));
+        $process = $this->getProcess(\sprintf('echo %s %s echo %s', $input, $operator, $input));
         $process->run();
         $this->assertEquals($expected, $process->getOutput());
     }
@@ -980,7 +980,7 @@ public function testMethodsThatNeedARunningProcess($method)
         $process = $this->getProcess('foo');
 
         $this->expectException(LogicException::class);
-        $this->expectExceptionMessage(sprintf('Process must be started before calling "%s()".', $method));
+        $this->expectExceptionMessage(\sprintf('Process must be started before calling "%s()".', $method));
 
         $process->{$method}();
     }
@@ -1475,7 +1475,7 @@ public function testEscapeArgument($arg)
 
     public function testRawCommandLine()
     {
-        $p = Process::fromShellCommandline(sprintf('"%s" -r %s "a" "" "b"', self::$phpBin, escapeshellarg('print_r($argv);')));
+        $p = Process::fromShellCommandline(\sprintf('"%s" -r %s "a" "" "b"', self::$phpBin, escapeshellarg('print_r($argv);')));
         $p->run();
 
         $expected = "Array\n(\n    [0] => -\n    [1] => a\n    [2] => \n    [3] => b\n)\n";
diff --git a/Tests/SignalListener.php b/Tests/SignalListener.php
@@ -9,7 +9,10 @@
  * file that was distributed with this source code.
  */
 
-pcntl_signal(\SIGUSR1, function () { echo 'SIGUSR1'; exit; });
+pcntl_signal(\SIGUSR1, function () {
+    echo 'SIGUSR1';
+    exit;
+});
 
 echo 'Caught ';
 

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ public function __construct(Process $process)`
`26`	`26`	`{`
`27`	`27`	`$this->process = $process;`
`28`	`28`
`29`		`- parent::__construct(sprintf('The process has been signaled with signal "%s".', $process->getTermSignal()));`
	`29`	`+ parent::__construct(\sprintf('The process has been signaled with signal "%s".', $process->getTermSignal()));`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`public function getProcess(): Process`
Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ public function find(string $name, ?string $default = null, array $extraDirs = [`
`74`	`74`	`$suffixes = $this->suffixes;`
`75`	`75`	`$suffixes = array_merge($suffixes, $pathExt ? explode(\PATH_SEPARATOR, $pathExt) : ['.exe', '.bat', '.cmd', '.com']);`
`76`	`76`	`}`
`77`		`- $suffixes = '' !== pathinfo($name, PATHINFO_EXTENSION) ? array_merge([''], $suffixes) : array_merge($suffixes, ['']);`
	`77`	`+ $suffixes = '' !== pathinfo($name, \PATHINFO_EXTENSION) ? array_merge([''], $suffixes) : array_merge($suffixes, ['']);`
`78`	`78`	`foreach ($suffixes as $suffix) {`
`79`	`79`	`foreach ($dirs as $dir) {`
`80`	`80`	`if ('' === $dir) {`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ public function write(mixed $input)`
`50`	`50`	`return;`
`51`	`51`	`}`
`52`	`52`	`if ($this->isClosed()) {`
`53`		`- throw new RuntimeException(sprintf('"%s" is closed.', static::class));`
	`53`	`+ throw new RuntimeException(\sprintf('"%s" is closed.', static::class));`
`54`	`54`	`}`
`55`	`55`	`$this->input[] = ProcessUtils::validateInput(__METHOD__, $input);`
`56`	`56`	`}`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public function __construct(string $script, ?string $cwd = null, ?array $env = n`
`52`	`52`
`53`	`53`	`public static function fromShellCommandline(string $command, ?string $cwd = null, ?array $env = null, mixed $input = null, ?float $timeout = 60): static`
`54`	`54`	`{`
`55`		`- throw new LogicException(sprintf('The "%s()" method cannot be called when using "%s".', __METHOD__, self::class));`
	`55`	`+ throw new LogicException(\sprintf('The "%s()" method cannot be called when using "%s".', __METHOD__, self::class));`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ public function __construct(array $command, ?string $cwd = null, ?array $env = n`
`75`	`75`
`76`	`76`	`public static function fromShellCommandline(string $command, ?string $cwd = null, ?array $env = null, mixed $input = null, ?float $timeout = 60): static`
`77`	`77`	`{`
`78`		`- throw new LogicException(sprintf('The "%s()" method cannot be called when using "%s".', __METHOD__, self::class));`
	`78`	`+ throw new LogicException(\sprintf('The "%s()" method cannot be called when using "%s".', __METHOD__, self::class));`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`public function start(?callable $callback = null, array $env = []): void`
Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ protected function write(): ?array`
`101`	`101`	`} elseif (!isset($this->inputBuffer[0])) {`
`102`	`102`	`if (!\is_string($input)) {`
`103`	`103`	`if (!\is_scalar($input)) {`
`104`		`- throw new InvalidArgumentException(sprintf('"%s" yielded a value of type "%s", but only scalars and stream resources are supported.', get_debug_type($this->input), get_debug_type($input)));`
	`104`	`+ throw new InvalidArgumentException(\sprintf('"%s" yielded a value of type "%s", but only scalars and stream resources are supported.', get_debug_type($this->input), get_debug_type($input)));`
`105`	`105`	`}`
`106`	`106`	`$input = (string) $input;`
`107`	`107`	`}`
Original file line number	Diff line number	Diff line change
`@@ -340,7 +340,7 @@ public function start(?callable $callback = null, array $env = [])`
`340`	`340`	`}`
`341`	`341`
`342`	`342`	`if (!is_dir($this->cwd)) {`
`343`		`- throw new RuntimeException(sprintf('The provided cwd "%s" does not exist.', $this->cwd));`
	`343`	`+ throw new RuntimeException(\sprintf('The provided cwd "%s" does not exist.', $this->cwd));`
`344`	`344`	`}`
`345`	`345`
`346`	`346`	`$process = @proc_open($commandline, $descriptors, $this->processPipes->pipes, $this->cwd, $envPairs, $this->options);`
`@@ -1197,7 +1197,7 @@ public function setOptions(array $options)`
`1197`	`1197`	`foreach ($options as $key => $value) {`
`1198`	`1198`	`if (!\in_array($key, $existingOptions)) {`
`1199`	`1199`	`$this->options = $defaultOptions;`
`1200`		`- throw new LogicException(sprintf('Invalid option "%s" passed to "%s()". Supported options are "%s".', $key, __METHOD__, implode('", "', $existingOptions)));`
	`1200`	`+ throw new LogicException(\sprintf('Invalid option "%s" passed to "%s()". Supported options are "%s".', $key, __METHOD__, implode('", "', $existingOptions)));`
`1201`	`1201`	`}`
`1202`	`1202`	`$this->options[$key] = $value;`
`1203`	`1203`	`}`
`@@ -1453,10 +1453,10 @@ private function doSignal(int $signal, bool $throwException): bool`
`1453`	`1453`	`}`
`1454`	`1454`
`1455`	`1455`	`if ('\\' === \DIRECTORY_SEPARATOR) {`
`1456`		`- exec(sprintf('taskkill /F /T /PID %d 2>&1', $pid), $output, $exitCode);`
	`1456`	`+ exec(\sprintf('taskkill /F /T /PID %d 2>&1', $pid), $output, $exitCode);`
`1457`	`1457`	`if ($exitCode && $this->isRunning()) {`
`1458`	`1458`	`if ($throwException) {`
`1459`		`- throw new RuntimeException(sprintf('Unable to kill the process (%s).', implode(' ', $output)));`
	`1459`	`+ throw new RuntimeException(\sprintf('Unable to kill the process (%s).', implode(' ', $output)));`
`1460`	`1460`	`}`
`1461`	`1461`
`1462`	`1462`	`return false;`
`@@ -1466,12 +1466,12 @@ private function doSignal(int $signal, bool $throwException): bool`
`1466`	`1466`	`$ok = @proc_terminate($this->process, $signal);`
`1467`	`1467`	`} elseif (\function_exists('posix_kill')) {`
`1468`	`1468`	`$ok = @posix_kill($pid, $signal);`
`1469`		`- } elseif ($ok = proc_open(sprintf('kill -%d %d', $signal, $pid), [2 => ['pipe', 'w']], $pipes)) {`
	`1469`	`+ } elseif ($ok = proc_open(\sprintf('kill -%d %d', $signal, $pid), [2 => ['pipe', 'w']], $pipes)) {`
`1470`	`1470`	`$ok = false === fgets($pipes[2]);`
`1471`	`1471`	`}`
`1472`	`1472`	`if (!$ok) {`
`1473`	`1473`	`if ($throwException) {`
`1474`		`- throw new RuntimeException(sprintf('Error while sending signal "%s".', $signal));`
	`1474`	`+ throw new RuntimeException(\sprintf('Error while sending signal "%s".', $signal));`
`1475`	`1475`	`}`
`1476`	`1476`
`1477`	`1477`	`return false;`
`@@ -1528,7 +1528,7 @@ function ($m) use (&$env, $uid) {`
`1528`	`1528`
`1529`	`1529`	`if (!$comSpec && $comSpec = (new ExecutableFinder())->find('cmd.exe')) {`
`1530`	`1530`	`// Escape according to CommandLineToArgvW rules`
`1531`		`- $comSpec = '"'.preg_replace('{(\\\\*+)"}', '$1$1\"', $comSpec) .'"';`
	`1531`	`+ $comSpec = '"'.preg_replace('{(\\\\*+)"}', '$1$1\"', $comSpec).'"';`
`1532`	`1532`	`}`
`1533`	`1533`
`1534`	`1534`	`$cmd = ($comSpec ?? 'cmd').' /V:ON /E:ON /D /C ('.str_replace("\n", ' ', $cmd).')';`
`@@ -1547,7 +1547,7 @@ function ($m) use (&$env, $uid) {`
`1547`	`1547`	`private function requireProcessIsStarted(string $functionName): void`
`1548`	`1548`	`{`
`1549`	`1549`	`if (!$this->isStarted()) {`
`1550`		`- throw new LogicException(sprintf('Process must be started before calling "%s()".', $functionName));`
	`1550`	`+ throw new LogicException(\sprintf('Process must be started before calling "%s()".', $functionName));`
`1551`	`1551`	`}`
`1552`	`1552`	`}`
`1553`	`1553`
`@@ -1559,7 +1559,7 @@ private function requireProcessIsStarted(string $functionName): void`
`1559`	`1559`	`private function requireProcessIsTerminated(string $functionName): void`
`1560`	`1560`	`{`
`1561`	`1561`	`if (!$this->isTerminated()) {`
`1562`		`- throw new LogicException(sprintf('Process must be terminated before calling "%s()".', $functionName));`
	`1562`	`+ throw new LogicException(\sprintf('Process must be terminated before calling "%s()".', $functionName));`
`1563`	`1563`	`}`
`1564`	`1564`	`}`
`1565`	`1565`
`@@ -1589,7 +1589,7 @@ private function replacePlaceholders(string $commandline, array $env): string`
`1589`	`1589`	`{`
`1590`	`1590`	`return preg_replace_callback('/"\$\{:([_a-zA-Z]++[_a-zA-Z0-9]*+)\}"/', function ($matches) use ($commandline, $env) {`
`1591`	`1591`	`if (!isset($env[$matches[1]]) \|\| false === $env[$matches[1]]) {`
`1592`		`- throw new InvalidArgumentException(sprintf('Command line is missing a value for parameter "%s": ', $matches[1]).$commandline);`
	`1592`	`+ throw new InvalidArgumentException(\sprintf('Command line is missing a value for parameter "%s": ', $matches[1]).$commandline);`
`1593`	`1593`	`}`
`1594`	`1594`
`1595`	`1595`	`return $this->escapeArgument($env[$matches[1]]);`