Fix off-by-one error in general Range iterator

We always skipped the first element. Noticed by dwarring++
rakudo · Aug 22, 2015 · bd5cfcb · bd5cfcb
1 parent 769c5b3
commit bd5cfcb
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/core/Range.pm b/src/core/Range.pm
@@ -179,8 +179,14 @@ my class Range is Cool does Iterable does Positional {
                 }
 
                 method pull-one() {
-                    $!i.=succ;
-                    $!i <= $!e ?? $!i !! IterationEnd
+                    if $!i <= $!e {
+                        my Mu $i = $!i;
+                        $!i = $i.succ;
+                        $i
+                    }
+                    else {
+                        IterationEnd
+                    }
                 }
             }.new($value, $!excludes-max ?? $!max.pred !! $!max)
         }