Support using arguments as an iterable.

We can't add an @@iterator field to the prototype of `arguments`, so we need to special case it in ES.GetIterator and ES.IsIterable instead.
paulmillr · Feb 14, 2014 · 7ea1664 · 7ea1664
1 parent 85dc5ed
commit 7ea1664
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/es6-shim.js b/es6-shim.js
@@ -160,10 +160,15 @@
       },
 
       IsIterable: function(o) {
-        return ES.TypeIsObject(o) && ES.IsCallable(o[$iterator$]);
+        return ES.TypeIsObject(o) &&
+          (ES.IsCallable(o[$iterator$]) || isArguments(o));
       },
 
       GetIterator: function(o) {
+        if (isArguments(o)) {
+          // special case support for `arguments`
+          return new ArrayIterator(o, "value");
+        }
         var it = o[$iterator$]();
         if (!ES.TypeIsObject(it)) {
           throw new TypeError('bad iterator');