Removed implicitly nullable arguments

Author: shudd3r

src/AppHandler.php

@@ -35,7 +35,7 @@ abstract class AppHandler implements RequestHandlerInterface
     /**
      * @param Build|null $build
      */
-    public function __construct(Build $build = null)
+    public function __construct(?Build $build = null)
     {
         $this->registerShutdown((bool) getenv(static::DEV_ENVIRONMENT));
         $this->setup = $this->environmentSetup($build ?? new Build());
@@ -56,7 +56,7 @@ final public function handle(ServerRequestInterface $request): ResponseInterface
     /**
      * @param string $id
      *
-     * @return Setup\Entry
+     * @return Entry
      */
     final public function config(string $id): Entry
     {

tests/Doubles/FakeServerRequest.php

@@ -24,7 +24,7 @@ class FakeServerRequest implements ServerRequestInterface
     public array        $cookies = [];
     public array        $parsed  = [];
 
-    public function __construct(string $method = 'GET', UriInterface $uri = null)
+    public function __construct(string $method = 'GET', ?UriInterface $uri = null)
     {
         $this->method = $method;
         $this->uri    = $uri ?? FakeUri::fromString('//example.com/foo/bar');

tests/Fixtures/HeadersState.php

@@ -34,7 +34,7 @@ public static function set(string $headerLine, $remove = true): void
         self::$headers[strtolower($name)][] = $headerLine;
     }
 
-    public static function remove(string $name = null): void
+    public static function remove(?string $name = null): void
     {
         if ($name === null) {
             self::$headers = [];

tests/Fixtures/shutdown-functions.php

@@ -18,7 +18,7 @@ function register_shutdown_function(callable $callback): void
     ShutdownState::$override ? ShutdownState::$callback = $callback : \register_shutdown_function($callback);
 }
 
-function http_response_code(int $code = null): void
+function http_response_code(?int $code = null): void
 {
     ShutdownState::$override ? ShutdownState::$status = $code : \http_response_code($code);
 }

tests/ServerProcessTest.php

@@ -91,7 +91,7 @@ public function testHeadersSetOutsideServerInstanceAreIgnored()
         $this->assertFalse(isset(Fixtures\HeadersState::$headers['x-powered-by']));
     }
 
-    private function server(Doubles\FakeResponse $response = null, int $buffer = 0): ServerProcess
+    private function server(?Doubles\FakeResponse $response = null, int $buffer = 0): ServerProcess
     {
         Fixtures\HeadersState::reset();
         $requestHandler = new Doubles\FakeRequestHandler(function () use ($response) {
@@ -100,11 +100,11 @@ private function server(Doubles\FakeResponse $response = null, int $buffer = 0):
         return new ServerProcess($requestHandler, $buffer);
     }
 
-    private function emit(ServerProcess $server, Doubles\FakeServerRequest $request = null): string
+    private function emit(ServerProcess $server): string
     {
         ob_start();
         try {
-            $server->execute($request ?: new Doubles\FakeServerRequest());
+            $server->execute(new Doubles\FakeServerRequest());
         } catch (RuntimeException $ex) {
             ob_get_clean();
             throw $ex;