gielfeldt / iterators

src/AtomicTempFileObject.php

Doc Comments +1 added lines, -3 removed lines

@@ -133,7 +133,7 @@ 
     /**
      * Easy access iterator apply for processing an entire file.
      *
-     * @param  Iterator $input    [description]
+     * @param  \Iterator $input    [description]
      * @param  callable $callback [description]
      */
     public function process(\Iterator $input, callable $callback)
@@ -169,8 +169,6 @@ 
     /**
      * File comparison
      *
-     * @param string $filename
-     *   The file to check against.
      *
      * @return bool
      *   True if the contents of this file matches the contents of $filename.

Spacing +3 added lines, -3 removed lines

@@ -140,7 +140,7 @@ 
     {
         $callback = \Closure::fromCallable($callback);
         $input->rewind();
-        iterator_apply($input, function (\Iterator $iterator) use ($callback) {
+        iterator_apply($input, function(\Iterator $iterator) use ($callback) {
             $callback($iterator->current(), $iterator->key(), $iterator, $this);
             return true;
         }, [$input]);
@@ -194,8 +194,8 @@ 
         // Rewind this temp file and compare it with the specified file.
         $identical = true;
         $tempFile->fseek(0);
-        while(!$file->eof()) {
-            if($file->fread(8192) != $tempFile->fread(8192)) {
+        while (!$file->eof()) {
+            if ($file->fread(8192) != $tempFile->fread(8192)) {
                 $identical = false;
                 break;
             }

Braces +3 added lines, -6 removed lines

@@ -115,15 +115,12 @@
         if ($this->persist == self::PERSIST || $this->persist == self::PERSIST_UNCHANGED) {
             if ($this->persist == self::PERSIST_UNCHANGED || !$this->doCompare()) {
                 $this->doPersist();
-            }
-            else {
+            } else {
                 $this->doDiscard();
             }
-        }
-        elseif ($this->persist & self::DISCARD) {
+        } elseif ($this->persist & self::DISCARD) {
             $this->doDiscard();
-        }
-        else {
+        } else {
             // @codeCoverageIgnoreStart
             trigger_error("Temp file left on device: " . $this->getRealPath(), E_USER_WARNING);
             // @codeCoverageIgnoreEnd

src/GlobStreamWrapper.php

Doc Comments +3 added lines

@@ -16,6 +16,9 @@
         self::$root = $root;
     }
 
+    /**
+     * @param string $path
+     */
     protected static function getRootedPath($path)
     {
         // Avoid infinite recursion.

src/Iterator.php

Doc Comments +3 added lines

@@ -30,6 +30,9 @@
         return $iterators ? end($iterators) : false;
     }
 
+    /**
+     * @return string
+     */
     public static function reduce(\Traversable $iterator, callable $callback, $initial = null)
     {
         $callback = \Closure::fromCallable($callback);

Spacing +7 added lines, -7 removed lines

@@ -33,7 +33,7 @@ 
     public static function reduce(\Traversable $iterator, callable $callback, $initial = null)
     {
         $callback = \Closure::fromCallable($callback);
-        iterator_apply($iterator, function ($iterator) use (&$initial, $callback) {
+        iterator_apply($iterator, function($iterator) use (&$initial, $callback) {
             $initial = $callback($initial, $iterator->current(), $iterator->key());
             return true;
         }, [$iterator]);
@@ -42,14 +42,14 @@ 
 
     public static function sum(\Traversable $iterator)
     {
-        return self::reduce($iterator, function ($carry, $value, $key) {
+        return self::reduce($iterator, function($carry, $value, $key) {
             return $carry + $value;
         }, 0);
     }
 
     public static function product(\Traversable $iterator)
     {
-        return self::reduce($iterator, function ($carry, $value, $key) {
+        return self::reduce($iterator, function($carry, $value, $key) {
             return $carry * $value;
         }, 1);
     }
@@ -61,28 +61,28 @@ 
 
     public static function min(\Traversable $iterator)
     {
-        return self::reduce($iterator, function ($carry, $value, $key) {
+        return self::reduce($iterator, function($carry, $value, $key) {
             return $carry < $value ? $carry : $value;
         }, INF);
     }
 
     public static function max(\Traversable $iterator)
     {
-        return self::reduce($iterator, function ($carry, $value, $key) {
+        return self::reduce($iterator, function($carry, $value, $key) {
             return $carry > $value ? $carry : $value;
         }, -INF);
     }
 
     public static function concatenate(\Traversable $iterator)
     {
-        return self::reduce($iterator, function ($carry, $value, $key) {
+        return self::reduce($iterator, function($carry, $value, $key) {
             return $carry . $value;
         }, '');
     }
 
     public static function implode($separator, \Traversable $iterator)
     {
-        $result = self::reduce($iterator, function ($carry, $value, $key) use ($separator) {
+        $result = self::reduce($iterator, function($carry, $value, $key) use ($separator) {
             return $carry . $value . $separator;
         }, '');
         $result = mb_substr($result, 0, -mb_strlen($separator));

Braces +1 added lines, -2 removed lines

@@ -96,8 +96,7 @@
             $value = $value instanceof \Traversable ? self::iterator_to_array_deep($value, $use_keys) : $value;
             if ($use_keys) {
                 $result[$key] = $value;
-            }
-            else {
+            } else {
                 $result[] = $value;
             }
         }

src/SortIterator.php

Doc Comments +3 added lines

@@ -27,6 +27,9 @@
         parent::__construct($this->getSortedIterator($iterator));
     }
 
+    /**
+     * @param \Traversable $iterator
+     */
     public function getSortedIterator($iterator)
     {
         $sortedIterator = new \ArrayIterator();

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
         $this->direction = $direction;
         $this->flags = $flags;
         $this->callback = \Closure::fromCallable($callback);
-        $this->realCallback = $direction == self::SORT_ASC ? $this->callback : function ($cmpA, $cmpB) {
+        $this->realCallback = $direction == self::SORT_ASC ? $this->callback : function($cmpA, $cmpB) {
             return ($this->callback)($cmpB, $cmpA);
         };
         parent::__construct($this->getSortedIterator($iterator));

src/UniqueIterator.php

Braces +1 added lines, -2 removed lines

@@ -18,8 +18,7 @@
 
         if ($flags & self::ASSUME_SORTED) {
             $this->method = \Closure::fromCallable([$this, 'filterAssumeSorted']);
-        }
-        else {
+        } else {
             $this->method = \Closure::fromCallable([$this, 'filterAssumeUnsorted']);
         }
     }

src/RepeatIterator.php

Braces +1 added lines, -2 removed lines

@@ -88,8 +88,7 @@
                 parent::rewind();
                 return $this->valid();
             }
-        }
-        elseif (
+        } elseif (
             isset($this->innerLimit) && $this->innerPos >= $this->innerLimit
             && $this->currentIteration == $this->count - 1
         ) {

src/DiffIterator.php

Indentation +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
             }
         }
         return true;
-	}
+    }
 
     public static function diffCurrent($iterator, $key, $value)
     {

src/MapIterator.php

Braces +2 added lines, -4 removed lines

@@ -23,14 +23,12 @@
                 if (is_numeric($this->currentKey) && intval($this->currentKey) >= $this->idx) {
                     $this->idx = intval($this->currentKey) + 1;
                 }
-            }
-            else {
+            } else {
                 $this->currentKey = $this->idx++;
                 $this->currentValue = $result;
             }
 
-        }
-        else {
+        } else {
             $this->currentKey = null;
             $this->currentValue = null;
         }

src/GlobIterator.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
         $realPath = $path ? $path : './';
         $realPath = rtrim($realPath, '/') . '/';
 
-        $flags = $flags &~ \FilesystemIterator::CURRENT_AS_PATHNAME;
+        $flags = $flags & ~ \FilesystemIterator::CURRENT_AS_PATHNAME;
         $flags |= \FilesystemIterator::KEY_AS_PATHNAME;
         $iterator = new \RecursiveDirectoryIterator($realPath, $flags);
 
@@ -57,7 +57,7 @@ 
         GlobIteratorFileInfo::setPath($iteratorId, $path, $realPath);
 
         // Actual glob filtering.
-        $fIterator = new \CallbackFilterIterator($rIterator, function (&$current, &$key, $iterator) use ($iteratorId, $regexPattern) {
+        $fIterator = new \CallbackFilterIterator($rIterator, function(&$current, &$key, $iterator) use ($iteratorId, $regexPattern) {
             GlobIteratorFileInfo::setIteratorId($iteratorId);
             $fileInfo = $current->getFileInfo(GlobIteratorFileInfo::class);
             if ($this->flags & \FilesystemIterator::CURRENT_AS_PATHNAME) {

Braces +2 added lines, -4 removed lines

@@ -62,15 +62,13 @@
             $fileInfo = $current->getFileInfo(GlobIteratorFileInfo::class);
             if ($this->flags & \FilesystemIterator::CURRENT_AS_PATHNAME) {
                 $current = $fileInfo->getPathname();
-            }
-            else {
+            } else {
                 $current = $fileInfo;
             }
 
             if ($this->flags & \FilesystemIterator::KEY_AS_FILENAME) {
                 $key = $fileInfo->getFilename();
-            }
-            else {
+            } else {
                 $key = $fileInfo->getPathname();
             }
             return preg_match($regexPattern, $fileInfo->getPathname());