Style fixes and docs

Author: Nyholm

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 ### Added
 
+- Add HttpClientPool client to leverage load balancing and fallback mechanism [see the documentation](http://docs.php-http.org/en/latest/components/client-common.html) for more details.
 - `PluginClientFactory` to create `PluginClient` instances.
 - Added new option 'delay' for `RetryPlugin`.
 - Added new option 'decider' for `RetryPlugin`.
@@ -56,10 +57,6 @@ $plugin = new RetryPlugin(['delay' => function(RequestInterface $request, Except
 
 ## 1.3.0 - 2016-10-16
 
-### Added
-
-- Add HttpClientPool client to leverage load balancing and fallback mechanism [see the documentation](http://docs.php-http.org/en/latest/components/client-common.html) for more details
-
 ### Changed
 
 - Fix Emulated Trait to use Http based promise which respect the HttpAsyncClient interface

src/HttpClientPool.php

@@ -13,13 +13,15 @@
  */
 abstract class HttpClientPool implements HttpAsyncClient, HttpClient
 {
-    /** @var HttpClientPoolItem[] */
+    /**
+     * @var HttpClientPoolItem[]
+     */
     protected $clientPool = [];
 
     /**
      * Add a client to the pool.
      *
-     * @param HttpClient|HttpAsyncClient $client
+     * @param HttpClient|HttpAsyncClient|HttpClientPoolItem $client
      */
     public function addHttpClient($client)
     {

src/HttpClientPoolItem.php

@@ -17,22 +17,29 @@
  */
 class HttpClientPoolItem implements HttpClient, HttpAsyncClient
 {
-    /** @var int Number of request this client is currently sending */
+    /**
+     * @var int Number of request this client is currently sending
+     */
     private $sendingRequestCount = 0;
 
-    /** @var \DateTime|null Time when this client has been disabled or null if enable */
+    /**
+     * @var \DateTime|null Time when this client has been disabled or null if enable
+     */
     private $disabledAt;
 
-    /** @var int|null Number of seconds after this client is reenable, by default null: never reenable this client */
+    /**
+     * @var int|null Number of seconds after this client is reenable, by default null: never reenable this client
+     */
     private $reenableAfter;
 
-    /** @var FlexibleHttpClient A http client responding to async and sync request */
+    /**
+     * @var FlexibleHttpClient A http client responding to async and sync request
+     */
     private $client;
 
     /**
-     * {@inheritdoc}
-     *
-     * @param null|int $reenableAfter Number of seconds after this client is reenable
+     * @param HttpClient|HttpAsyncClient $client
+     * @param null|int                   $reenableAfter Number of seconds after this client is reenable
      */
     public function __construct($client, $reenableAfter = null)
     {
@@ -91,6 +98,8 @@ public function sendAsyncRequest(RequestInterface $request)
      *
      * Will also reactivate this client if possible
      *
+     * @internal
+     *
      * @return bool
      */
     public function isDisabled()
@@ -116,6 +125,8 @@ public function isDisabled()
     /**
      * Get current number of request that is send by the underlying http client.
      *
+     * @internal
+     *
      * @return int
      */
     public function getSendingRequestCount()