Cleanup newly detected dead code

Author: staabm

src/Command/AnalyseCommand.php

@@ -746,9 +746,6 @@ private function generateBaseline(string $generateBaselineFile, InceptionResult
 		$stream = $streamOutput->getStream();
 		rewind($stream);
 		$baselineContents = stream_get_contents($stream);
-		if ($baselineContents === false) {
-			throw new ShouldNotHappenException();
-		}
 
 		try {
 			DirectoryCreator::ensureDirectoryExists($baselineFileDirectory, 0644);

src/Parallel/Process.php

@@ -9,7 +9,6 @@
 use React\Stream\WritableStreamInterface;
 use Throwable;
 use function fclose;
-use function is_string;
 use function rewind;
 use function sprintf;
 use function stream_get_contents;
@@ -73,16 +72,11 @@ public function start(callable $onData, callable $onError, callable $onExit): vo
 
 			$output = '';
 			rewind($this->stdOut);
-			$stdOut = stream_get_contents($this->stdOut);
-			if (is_string($stdOut)) {
-				$output .= $stdOut;
-			}
+			$output .= stream_get_contents($this->stdOut);
 
 			rewind($this->stdErr);
-			$stdErr = stream_get_contents($this->stdErr);
-			if (is_string($stdErr)) {
-				$output .= $stdErr;
-			}
+			$output .= stream_get_contents($this->stdErr);
+
 			$onExit($exitCode, $output);
 			fclose($this->stdOut);
 			fclose($this->stdErr);

src/Process/ProcessPromise.php

@@ -69,9 +69,6 @@ public function run(): PromiseInterface
 			}
 
 			if ($exitCode === 0) {
-				if ($stdOut === false) {
-					$stdOut = '';
-				}
 				$this->deferred->resolve($stdOut);
 				return;
 			}

src/Testing/ErrorFormatterTestCase.php

@@ -75,10 +75,6 @@ protected function getOutputContent(bool $decorated = false, bool $verbose = fal
 		rewind($this->getOutputStream($decorated, $verbose)->getStream());
 
 		$contents = stream_get_contents($this->getOutputStream($decorated, $verbose)->getStream());
-		if ($contents === false) {
-			throw new ShouldNotHappenException();
-		}
-
 		return $this->rtrimMultiline($contents);
 	}
 

tests/PHPStan/Command/AnalyseApplicationIntegrationTest.php

@@ -93,10 +93,6 @@ private function runPath(string $path, int $expectedStatusCode): string
 		rewind($output->getStream());
 
 		$contents = stream_get_contents($output->getStream());
-		if ($contents === false) {
-			throw new ShouldNotHappenException();
-		}
-
 		$this->assertSame($expectedStatusCode, $statusCode, $contents);
 
 		return $contents;

tests/PHPStan/Command/CommandHelperTest.php

@@ -136,19 +136,13 @@ public function testBegin(
 			if (!$expectException) {
 				rewind($output->getStream());
 				$contents = stream_get_contents($output->getStream());
-				if ($contents === false) {
-					throw new ShouldNotHappenException();
-				}
 				$this->fail($contents);
 			}
 		}
 
 		rewind($output->getStream());
 
 		$contents = stream_get_contents($output->getStream());
-		if ($contents === false) {
-			throw new ShouldNotHappenException();
-		}
 		$this->assertStringContainsString($expectedOutput, $contents);
 
 		if (isset($result)) {

tests/PHPStan/Command/ErrorFormatter/BaselineNeonErrorFormatterTest.php

@@ -437,10 +437,6 @@ public function testEndOfFileNewlines(
 		rewind($outputStream->getStream());
 
 		$content = stream_get_contents($outputStream->getStream());
-		if ($content === false) {
-			throw new ShouldNotHappenException();
-		}
-
 		if ($expectedNewlinesCount > 0) {
 			Assert::assertSame(str_repeat("\n", $expectedNewlinesCount), substr($content, -$expectedNewlinesCount));
 		}