From: gggeek <giunta.gaetano@gmail.com>
Date: Sun, 12 Jul 2015 10:40:27 +0000 (+0100)
Subject: Docs and formatting to please Scrutinizer
X-Git-Tag: 4.0.0-alpha^2~10
X-Git-Url: http://git.onelab.eu/?p=plcapi.git;a=commitdiff_plain;h=22592a7334c7ae95b074c4e967cf18e519284f40

Docs and formatting to please Scrutinizer
---

diff --git a/pakefile.php b/pakefile.php
index 30acead..74bcba8 100644
--- a/pakefile.php
+++ b/pakefile.php
@@ -78,11 +78,19 @@ class Builder
         //pake_echo('---'.self::$libVersion.'---');
     }
 
+    /**
+     * @param string $name
+     * @return string
+     */
     public static function tool($name)
     {
         return self::$tools[$name];
     }
 
+    /**
+     * @param string $name
+     * @return string
+     */
     public static function option($name)
     {
         return self::$options[$name];
diff --git a/src/Server.php b/src/Server.php
index 8540553..d530fcd 100644
--- a/src/Server.php
+++ b/src/Server.php
@@ -347,7 +347,7 @@ class Server
     /**
      * Parse http headers received along with xmlrpc request. If needed, inflate request.
      *
-     * @return mixed null on success or a Response
+     * @return mixed Response|null on success or an error Response
      */
     protected function parseRequestHeaders(&$data, &$reqEncoding, &$respEncoding, &$respCompression)
     {
diff --git a/src/Wrapper.php b/src/Wrapper.php
index 10aae52..7a8af5e 100644
--- a/src/Wrapper.php
+++ b/src/Wrapper.php
@@ -175,8 +175,7 @@ class Wrapper
 
             $plainFuncName = 'Closure';
             $exists = true;
-        }
-        else {
+        } else {
             $plainFuncName = $callable;
             $exists = function_exists($callable);
         }
@@ -397,7 +396,7 @@ class Wrapper
      * @param array $extraOptions
      * @param string $plainFuncName
      * @param string $funcDesc
-     * @return callable
+     * @return \Closure
      */
     protected function buildWrapFunctionClosure($callable, $extraOptions, $plainFuncName, $funcDesc)
     {
@@ -498,7 +497,7 @@ class Wrapper
      * @param array $extraOptions
      * @param string $plainFuncName
      * @param array $funcDesc
-     * @return array
+     * @return string
      *
      * @todo add a nice phpdoc block in the generated source
      */
@@ -786,7 +785,7 @@ class Wrapper
      * @param string $methodName
      * @param array $extraOptions
      * @param string $mSig
-     * @return callable
+     * @return \Closure
      *
      * @todo should we allow usage of parameter simple_client_copy to mean 'do not clone' in this case?
      */