« Bugzilla Issues Index
#2672 — 7.4.1 GetIterator, 7.4.2 IsIterable: Missing IsCallable checks and typos
- bug_id:
2672
- creation_ts:
2014-04-17 06:32:00 -0700
- short_desc:
7.4.1 GetIterator, 7.4.2 IsIterable: Missing IsCallable checks and typos
- delta_ts:
2014-05-06 16:56:48 -0700
- product:
Draft for 6th Edition
- component:
technical issue
- version:
Rev 23: April 5, 2014 Draft
- rep_platform:
All
- op_sys:
All
- bug_status:
RESOLVED
- resolution:
FIXED
- priority:
Normal
- bug_severity:
normal
- everconfirmed:
true
- reporter:
André Bargull
- assigned_to:
Allen Wirfs-Brock
- commentid:
7771
- comment_count:
0
- who:
André Bargull
- bug_when:
2014-04-17 06:32:17 -0700
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"
- commentid:
7787
- comment_count:
1
- who:
Allen Wirfs-Brock
- bug_when:
2014-04-17 09:00:13 -0700
fixed in rev24 editor's draft
renamed IsIterable -> CheckIterable
- commentid:
8194
- comment_count:
2
- who:
Allen Wirfs-Brock
- bug_when:
2014-05-06 16:56:48 -0700
fixed in rev24