Docs and formatting to please Scrutinizer

diff --git a/pakefile.php b/pakefile.php
index 30acead..74bcba8 100644 (file)
--- a/pakefile.php
+++ b/pakefile.php
@@ -78,11 +78,19 @@ class Builder
         //pake_echo('---'.self::$libVersion.'---');
     }
 
         //pake_echo('---'.self::$libVersion.'---');
     }
 

+    /**
+     * @param string $name
+     * @return string
+     */

     public static function tool($name)
     {
         return self::$tools[$name];
     }
 
     public static function tool($name)
     {
         return self::$tools[$name];
     }
 

+    /**
+     * @param string $name
+     * @return string
+     */

     public static function option($name)
     {
         return self::$options[$name];
     public static function option($name)
     {
         return self::$options[$name];

diff --git a/src/Server.php b/src/Server.php
index 8540553..d530fcd 100644 (file)
--- 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.
      *
     /**
      * 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)
     {
      */
     protected function parseRequestHeaders(&$data, &$reqEncoding, &$respEncoding, &$respCompression)
     {

diff --git a/src/Wrapper.php b/src/Wrapper.php
index 10aae52..7a8af5e 100644 (file)
--- a/src/Wrapper.php
+++ b/src/Wrapper.php
@@ -175,8 +175,7 @@ class Wrapper
 
             $plainFuncName = 'Closure';
             $exists = true;
 
             $plainFuncName = 'Closure';
             $exists = true;

-        }
-        else {
+        } else {

             $plainFuncName = $callable;
             $exists = function_exists($callable);
         }
             $plainFuncName = $callable;
             $exists = function_exists($callable);
         }


@@ -397,7 +396,7 @@ class Wrapper
      * @param array $extraOptions
      * @param string $plainFuncName
      * @param string $funcDesc
      * @param array $extraOptions
      * @param string $plainFuncName
      * @param string $funcDesc

-     * @return callable
+     * @return \Closure

      */
     protected function buildWrapFunctionClosure($callable, $extraOptions, $plainFuncName, $funcDesc)
     {
      */
     protected function buildWrapFunctionClosure($callable, $extraOptions, $plainFuncName, $funcDesc)
     {


@@ -498,7 +497,7 @@ class Wrapper
      * @param array $extraOptions
      * @param string $plainFuncName
      * @param array $funcDesc
      * @param array $extraOptions
      * @param string $plainFuncName
      * @param array $funcDesc

-     * @return array
+     * @return string

      *
      * @todo add a nice phpdoc block in the generated source
      */
      *
      * @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
      * @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?
      */
      *
      * @todo should we allow usage of parameter simple_client_copy to mean 'do not clone' in this case?
      */