archives

« Bugzilla Issues Index

#2672 — 7.4.1 GetIterator, 7.4.2 IsIterable: Missing IsCallable checks and typos


7.4.1 GetIterator only checks for IsCallable in step 1.c if the method parameter was not passed, that means if method was passed no such check is performed.

Either make step 1.c a new step between step 1 and step 2, or change 7.4.2 IsIterable to use GetMethod (and then handle only 'undefined' in 7.4.1 step 1.c).

Also "IsIterable" should probably be renamed to "GetIteratorMethod" are something similar, the "Is*"-prefix is normally only used for methods which return boolean results (e.g. IsCallable, IsConstructor, IsExtensible).


Typos:
- 7.4.1, step 2: Remove "and."
- 7.4.2, step 2: "@@Iterator" -> "@@iterator"


fixed in rev24 editor's draft

renamed IsIterable -> CheckIterable


fixed in rev24