Make traversable wrapping only apply when necessary

src/Fusonic/Linq/Linq.php

@@ -119,9 +119,13 @@ public function skip($count)
                 return new Linq([]);
             }
         }
+        if ($innerIterator instanceof \Iterator === false) {
+            // IteratorIterator wraps $innerIterator because it is Traversable but not an Iterator.
+            // (see https://bugs.php.net/bug.php?id=52280)
+            $innerIterator = new \IteratorIterator($innerIterator);
+        }
 
-        // IteratorIterator wraps $innerIterator because it might be Traversable but not an Iterator.
-        return new Linq(new \LimitIterator(new \IteratorIterator($innerIterator), $count, -1));
+        return new Linq(new \LimitIterator($innerIterator, $count, -1));
     }
 
     /**
@@ -135,9 +139,14 @@ public function take($count)
         if ($count == 0) {
             return new Linq([]);
         }
+        $innerIterator = $this->iterator;
+        if ($innerIterator instanceof \Iterator === false) {
+            // IteratorIterator wraps $this->iterator because it is Traversable but not an Iterator.
+            // (see https://bugs.php.net/bug.php?id=52280)
+            $innerIterator = new \IteratorIterator($innerIterator);
+        }
 
-        // IteratorIterator wraps $this->iterator because it might be Traversable but not an Iterator.
-        return new Linq(new \LimitIterator(new \IteratorIterator($this->iterator), 0, $count));
+        return new Linq(new \LimitIterator($innerIterator, 0, $count));
     }
 
     /**

tests/Fusonic/Linq/Test/SkipTakeTest.php

@@ -1,7 +1,7 @@
 <?php
 
 require_once("TestBase.php");
-require_once __DIR__ . '/TestIteratorAggregate.php';
+require_once(__DIR__ . "/TestIteratorAggregate.php");
 
 use Fusonic\Linq\Linq;