zicht / itertools

src/Zicht/Itertools/lib/Traits/ChainTrait.php

Doc Comments -2 removed lines

@@ -26,8 +26,6 @@
      * > iter\iterable('ABC', 'DEF')->chain()
      * A B C D E F
      *
-     * @param array|string|\Iterator $iterable
-     * @param array|string|\Iterator $iterable2
      * @return iter\lib\ChainIterator
      */
     public function chain(/* $iterable, $iterable2, ... */)

src/Zicht/Itertools/lib/Traits/MapTrait.php

Doc Comments -1 removed lines

@@ -34,7 +34,6 @@
      * 2.5 3.5 4.5
      *
      * @param null|string|\Closure $strategy
-     * @param array|string|\Iterator $iterable2
      * @return iter\lib\MapIterator
      */
     public function map($strategy /*, $iterable2, ... */)

src/Zicht/Itertools/lib/Traits/ReduceTrait.php

Doc Comments -1 removed lines

@@ -25,7 +25,6 @@
      * > iter\iterable([])->reduce('min', 1)
      * 1
      *
-     * @param array|string|\Iterator $iterable
      * @param string|\Closure $closure
      * @param mixed $initializer
      * @return mixed

src/Zicht/Itertools/lib/Traits/ZipTrait.php

Doc Comments -1 removed lines

@@ -22,7 +22,6 @@
      * > zip([1, 2, 3], ['a', 'b', 'c'])
      * [1, 'a'] [2, 'b'] [3, 'c']
      *
-     * @param array|string|\Iterator $iterable2
      * @return iter\lib\ZipIterator
      */
     public function zip(/* $iterable2, ... */)

src/Zicht/Itertools/lib/GroupbyIterator.php

Spacing +1 added lines, -1 removed lines

@@ -160,7 +160,7 @@
         $array = iterator_to_array($this);
         array_walk(
             $array,
-            function (&$value) {
+            function(&$value) {
                 /** @var GroupedIterator $value */
                 $value = $value->toArray();
             }

src/Zicht/Itertools/util/Mappings.php

Spacing +4 added lines, -4 removed lines

@@ -16,7 +16,7 @@ 
      */
     public static function lstrip($chars = " \t\n\r\0\x0B")
     {
-        return function ($value) use ($chars) {
+        return function($value) use ($chars) {
             return ltrim($value, $chars);
         };
     }
@@ -26,7 +26,7 @@ 
      */
     public static function rstrip($chars = " \t\n\r\0\x0B")
     {
-        return function ($value) use ($chars) {
+        return function($value) use ($chars) {
             return rtrim($value, $chars);
         };
     }
@@ -36,7 +36,7 @@ 
      */
     public static function strip($chars = " \t\n\r\0\x0B")
     {
-        return function ($value) use ($chars) {
+        return function($value) use ($chars) {
             return trim($value, $chars);
         };
     }
@@ -46,7 +46,7 @@ 
      */
     public static function length()
     {
-        return function ($value) {
+        return function($value) {
             return sizeof($value);
         };
     }

src/Zicht/Itertools/reductions.php

Spacing +7 added lines, -7 removed lines

@@ -15,7 +15,7 @@ 
  */
 function add()
 {
-    return function ($a, $b) {
+    return function($a, $b) {
         if (!is_numeric($a)) {
             throw new \InvalidArgumentException('Argument $A must be numeric to perform addition');
         }
@@ -33,7 +33,7 @@ 
  */
 function sub()
 {
-    return function ($a, $b) {
+    return function($a, $b) {
         if (!is_numeric($a)) {
             throw new \InvalidArgumentException('Argument $A must be numeric to perform subtraction');
         }
@@ -51,7 +51,7 @@ 
  */
 function mul()
 {
-    return function ($a, $b) {
+    return function($a, $b) {
         if (!is_numeric($a)) {
             throw new \InvalidArgumentException('Argument $A must be numeric to perform multiplication');
         }
@@ -69,7 +69,7 @@ 
  */
 function min()
 {
-    return function ($a, $b) {
+    return function($a, $b) {
         if (!is_numeric($a)) {
             throw new \InvalidArgumentException('Argument $A must be numeric to determine minimum');
         }
@@ -87,7 +87,7 @@ 
  */
 function max()
 {
-    return function ($a, $b) {
+    return function($a, $b) {
         if (!is_numeric($a)) {
             throw new \InvalidArgumentException('Argument $A must be numeric to determine maximum');
         }
@@ -109,7 +109,7 @@ 
     if (!is_string($glue)) {
         throw new \InvalidArgumentException('Argument $GLUE must be a string to join');
     }
-    return function ($a, $b) use ($glue) {
+    return function($a, $b) use ($glue) {
         if (!is_string($a)) {
             throw new \InvalidArgumentException('Argument $A must be a string to join');
         }
@@ -131,7 +131,7 @@ 
  */
 function chain()
 {
-    return function ($chainIterator, $b) {
+    return function($chainIterator, $b) {
         if (!($chainIterator instanceof ChainIterator)) {
             throw new \InvalidArgumentException('Argument $A must be a ChainIterator.  Did your call "reduce" with "new ChainIterator()" as the initial parameter?');
         }

Doc Comments +1 added lines

@@ -176,6 +176,7 @@
 
 /**
  * @deprecated please use the reduction functions directly, will be removed in version 3.0
+ * @param string $name
  */
 function getReduction($name)
 {

src/Zicht/Itertools/mappings.php

Spacing +10 added lines, -10 removed lines

@@ -16,7 +16,7 @@ 
  */
 function lstrip($chars = " \t\n\r\0\x0B")
 {
-    return function ($value) use ($chars) {
+    return function($value) use ($chars) {
         return ltrim($value, $chars);
     };
 }
@@ -29,7 +29,7 @@ 
  */
 function rstrip($chars = " \t\n\r\0\x0B")
 {
-    return function ($value) use ($chars) {
+    return function($value) use ($chars) {
         return rtrim($value, $chars);
     };
 }
@@ -42,7 +42,7 @@ 
  */
 function strip($chars = " \t\n\r\0\x0B")
 {
-    return function ($value) use ($chars) {
+    return function($value) use ($chars) {
         return trim($value, $chars);
     };
 }
@@ -54,7 +54,7 @@ 
  */
 function length()
 {
-    return function ($value) {
+    return function($value) {
         if (is_null($value)) {
             return 0;
         }
@@ -74,7 +74,7 @@ 
  */
 function key()
 {
-    return function ($value, $key) {
+    return function($value, $key) {
         return $key;
     };
 }
@@ -86,7 +86,7 @@ 
  */
 function lower()
 {
-    return function ($value) {
+    return function($value) {
         return strtolower($value);
     };
 }
@@ -98,7 +98,7 @@ 
  */
 function upper()
 {
-    return function ($value) {
+    return function($value) {
         return strtoupper($value);
     };
 }
@@ -134,7 +134,7 @@ 
 {
     $strategies = array_map('\Zicht\Itertools\conversions\mixed_to_value_getter', $strategies);
 
-    return function ($value, $key) use ($strategies) {
+    return function($value, $key) use ($strategies) {
         $res = [];
         foreach ($strategies as $strategyKey => $strategy) {
             $res[$strategyKey] = $strategy($value, $key);
@@ -156,7 +156,7 @@ 
         $max = getrandmax();
     }
 
-    return function () use ($min, $max) {
+    return function() use ($min, $max) {
         return rand($min, $max);
     };
 }
@@ -170,7 +170,7 @@ 
 function type($strategy = null)
 {
     $strategy = conversions\mixed_to_value_getter($strategy);
-    return function ($value) use ($strategy) {
+    return function($value) use ($strategy) {
         $value = $strategy($value);
         return is_object($value) ? get_class($value) : gettype($value);
     };

src/Zicht/Itertools/conversions.php

Spacing +3 added lines, -3 removed lines

@@ -69,7 +69,7 @@ 
 function mixed_to_closure($closure)
 {
     if (is_null($closure)) {
-        return function ($value) {
+        return function($value) {
             return $value;
         };
     }
@@ -79,7 +79,7 @@ 
         // A \Closure is always callable, but a callable is not always a \Closure.
         // Checking within this if statement is a slight optimization, preventing an unnecessary function wrap
         if (is_callable($closure)) {
-            $closure = function () use ($closure) {
+            $closure = function() use ($closure) {
                 return call_user_func_array($closure, func_get_args());
             };
         } else {
@@ -111,7 +111,7 @@ 
 {
     if (is_string($strategy)) {
         $keyParts = explode('.', $strategy);
-        $strategy = function ($value) use ($keyParts) {
+        $strategy = function($value) use ($keyParts) {
             foreach ($keyParts as $keyPart) {
                 if (is_object($value)) {
                     // property_exists does not distinguish between public, protected, or private properties, hence we need to use reflection