Fix code style with PHP-CS-Fixer

Author: Михаил Красильников

src/Client.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace Http\Client\Curl;
 
 use Http\Client\Exception;
@@ -13,61 +14,61 @@
 use Psr\Http\Message\ResponseInterface;
 
 /**
- * PSR-7 compatible cURL based HTTP client
+ * PSR-7 compatible cURL based HTTP client.
  *
  * @license http://opensource.org/licenses/MIT MIT
- *
  * @author  Михаил Красильников <m.krasilnikov@yandex.ru>
  * @author  Blake Williams <github@shabbyrobe.org>
  *
  * @api
+ *
  * @since   1.0
  */
 class Client implements HttpClient, HttpAsyncClient
 {
     /**
-     * cURL options
+     * cURL options.
      *
      * @var array
      */
     private $options;
 
     /**
-     * PSR-7 message factory
+     * PSR-7 message factory.
      *
      * @var MessageFactory
      */
     private $messageFactory;
 
     /**
-     * PSR-7 stream factory
+     * PSR-7 stream factory.
      *
      * @var StreamFactory
      */
     private $streamFactory;
 
     /**
-     * cURL synchronous requests handle
+     * cURL synchronous requests handle.
      *
      * @var resource|null
      */
     private $handle = null;
 
     /**
-     * Simultaneous requests runner
+     * Simultaneous requests runner.
      *
      * @var MultiRunner|null
      */
     private $multiRunner = null;
 
     /**
-     * Create new client
+     * Create new client.
      *
      * @param MessageFactory|null $messageFactory HTTP Message factory
      * @param StreamFactory|null  $streamFactory  HTTP Stream factory
      * @param array               $options        cURL options (see http://php.net/curl_setopt)
      *
-     * @throws \Http\Discovery\Exception\NotFoundException If factory discovery failed.
+     * @throws \Http\Discovery\Exception\NotFoundException If factory discovery failed
      *
      * @since 1.0
      */
@@ -82,7 +83,7 @@ public function __construct(
     }
 
     /**
-     * Release resources if still active
+     * Release resources if still active.
      */
     public function __destruct()
     {
@@ -98,13 +99,13 @@ public function __destruct()
      *
      * @return ResponseInterface
      *
-     * @throws \Http\Client\Exception\NetworkException In case of network problems.
-     * @throws \Http\Client\Exception\RequestException On invalid request.
-     * @throws \InvalidArgumentException For invalid header names or values.
-     * @throws \RuntimeException         If creating the body stream fails.
+     * @throws \Http\Client\Exception\NetworkException In case of network problems
+     * @throws \Http\Client\Exception\RequestException On invalid request
+     * @throws \InvalidArgumentException               For invalid header names or values
+     * @throws \RuntimeException                       If creating the body stream fails
      *
-     * @since 1.6 \UnexpectedValueException replaced with RequestException.
-     * @since 1.6 Throw NetworkException on network errors.
+     * @since 1.6 \UnexpectedValueException replaced with RequestException
+     * @since 1.6 Throw NetworkException on network errors
      * @since 1.0
      */
     public function sendRequest(RequestInterface $request)
@@ -149,11 +150,11 @@ public function sendRequest(RequestInterface $request)
      *
      * @return Promise
      *
-     * @throws \Http\Client\Exception\RequestException On invalid request.
-     * @throws \InvalidArgumentException For invalid header names or values.
-     * @throws \RuntimeException If creating the body stream fails.
+     * @throws \Http\Client\Exception\RequestException On invalid request
+     * @throws \InvalidArgumentException               For invalid header names or values
+     * @throws \RuntimeException                       If creating the body stream fails
      *
-     * @since 1.6 \UnexpectedValueException replaced with RequestException.
+     * @since 1.6 \UnexpectedValueException replaced with RequestException
      * @since 1.0
      */
     public function sendAsyncRequest(RequestInterface $request)
@@ -175,14 +176,14 @@ public function sendAsyncRequest(RequestInterface $request)
     }
 
     /**
-     * Generates cURL options
+     * Generates cURL options.
      *
      * @param RequestInterface $request
      * @param ResponseBuilder  $responseBuilder
      *
-     * @throws \Http\Client\Exception\RequestException On invalid request.
-     * @throws \InvalidArgumentException For invalid header names or values.
-     * @throws \RuntimeException if can not read body
+     * @throws \Http\Client\Exception\RequestException On invalid request
+     * @throws \InvalidArgumentException               For invalid header names or values
+     * @throws \RuntimeException                       if can not read body
      *
      * @return array
      */
@@ -231,7 +232,7 @@ private function createCurlOptions(RequestInterface $request, ResponseBuilder $r
     }
 
     /**
-     * Return cURL constant for specified HTTP version
+     * Return cURL constant for specified HTTP version.
      *
      * @param string $requestVersion
      *
@@ -311,7 +312,7 @@ private function addRequestBodyOptions(RequestInterface $request, array $options
     }
 
     /**
-     * Create headers array for CURLOPT_HTTPHEADER
+     * Create headers array for CURLOPT_HTTPHEADER.
      *
      * @param RequestInterface $request
      * @param array            $options cURL options
@@ -338,7 +339,7 @@ private function createHeaders(RequestInterface $request, array $options)
                 }
             }
             foreach ($values as $value) {
-                $curlHeaders[] = $name . ': ' . $value;
+                $curlHeaders[] = $name.': '.$value;
             }
         }
         /*
@@ -351,11 +352,11 @@ private function createHeaders(RequestInterface $request, array $options)
     }
 
     /**
-     * Create new ResponseBuilder instance
+     * Create new ResponseBuilder instance.
      *
      * @return ResponseBuilder
      *
-     * @throws \RuntimeException If creating the stream from $body fails.
+     * @throws \RuntimeException If creating the stream from $body fails
      */
     private function createResponseBuilder()
     {

src/CurlPromise.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace Http\Client\Curl;
 
 use Http\Promise\Promise;
@@ -12,20 +13,19 @@
  * Reason is replaced by an object where its class implement a Http\Client\Exception.
  *
  * @license http://opensource.org/licenses/MIT MIT
- *
  * @author  Михаил Красильников <m.krasilnikov@yandex.ru>
  */
 class CurlPromise implements Promise
 {
     /**
-     * Shared promise core
+     * Shared promise core.
      *
      * @var PromiseCore
      */
     private $core;
 
     /**
-     * Requests runner
+     * Requests runner.
      *
      * @var MultiRunner
      */
@@ -49,13 +49,13 @@ public function __construct(PromiseCore $core, MultiRunner $runner)
      * If you do not care about one of the cases, you can set the corresponding callable to null
      * The callback will be called when the response or exception arrived and never more than once.
      *
-     * @param callable $onFulfilled Called when a response will be available.
+     * @param callable $onFulfilled Called when a response will be available
      * @param callable $onRejected  Called when an error happens.
      *
-     * You must always return the Response in the interface or throw an Exception.
+     * You must always return the Response in the interface or throw an Exception
      *
      * @return Promise Always returns a new promise which is resolved with value of the executed
-     *                 callback (onFulfilled / onRejected).
+     *                 callback (onFulfilled / onRejected)
      */
     public function then(callable $onFulfilled = null, callable $onRejected = null)
     {
@@ -90,7 +90,7 @@ public function getState()
      *
      * @return \Psr\Http\Message\ResponseInterface|null Resolved value, null if $unwrap is set to false
      *
-     * @throws \Http\Client\Exception The rejection reason.
+     * @throws \Http\Client\Exception The rejection reason
      */
     public function wait($unwrap = true)
     {
@@ -103,6 +103,7 @@ public function wait($unwrap = true)
 
             return $this->core->getResponse();
         }
+
         return null;
     }
 }

src/MultiRunner.php

@@ -1,33 +1,33 @@
 <?php
+
 namespace Http\Client\Curl;
 
 use Http\Client\Exception\RequestException;
 
 /**
- * Simultaneous requests runner
+ * Simultaneous requests runner.
  *
  * @license http://opensource.org/licenses/MIT MIT
- *
  * @author  Михаил Красильников <m.krasilnikov@yandex.ru>
  */
 class MultiRunner
 {
     /**
-     * cURL multi handle
+     * cURL multi handle.
      *
      * @var resource|null
      */
     private $multiHandle = null;
 
     /**
-     * Awaiting cores
+     * Awaiting cores.
      *
      * @var PromiseCore[]
      */
     private $cores = [];
 
     /**
-     * Release resources if still active
+     * Release resources if still active.
      */
     public function __destruct()
     {
@@ -37,7 +37,7 @@ public function __destruct()
     }
 
     /**
-     * Add promise to runner
+     * Add promise to runner.
      *
      * @param PromiseCore $core
      */
@@ -58,7 +58,7 @@ public function add(PromiseCore $core)
     }
 
     /**
-     * Remove promise from runner
+     * Remove promise from runner.
      *
      * @param PromiseCore $core
      */
@@ -68,6 +68,7 @@ public function remove(PromiseCore $core)
             if ($existed === $core) {
                 curl_multi_remove_handle($this->multiHandle, $core->getHandle());
                 unset($this->cores[$index]);
+
                 return;
             }
         }
@@ -122,6 +123,7 @@ private function findCoreByHandle($handle)
                 return $core;
             }
         }
+
         return null;
     }
 }