« Bugzilla Issues Index
#3780 — 21.1.3.17 String.prototype.split: Move ToString to top ?
- bug_id:
3780
- creation_ts:
2015-02-05 14:29:00 -0800
- short_desc:
21.1.3.17 String.prototype.split: Move ToString to top ?
- delta_ts:
2015-02-12 12:17:39 -0800
- product:
Draft for 6th Edition
- component:
technical issue
- version:
Rev 32: February 2, 2015 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:
12151
- comment_count:
0
- who:
André Bargull
- bug_when:
2015-02-05 14:29:06 -0800
21.1.3.17 String.prototype.split ( separator, limit )
Steps 4-5.
Move after step 1 and remove step 2?
> 1. Let O be RequireObjectCoercible(this value).
> 2. Let S be ToString(O).
> 3. ReturnIfAbrupt(S).
And update step 3.c.i to use `S` instead of `O`.
Cf.
1.1.3.11 String.prototype.match ( regexp )
21.1.3.14 String.prototype.replace (searchValue, replaceValue )
21.1.3.15 String.prototype.search ( regexp )
- commentid:
12174
- comment_count:
1
- who:
Allen Wirfs-Brock
- bug_when:
2015-02-05 17:16:07 -0800
fixed in rev33 editor's draft
what I actually ended up doing is changed match/replace/search to use the pattern from split.
It seems better to let the delegated method do the conversion (which it needs to do anyway, just in case somebody else calls them)
- commentid:
12431
- comment_count:
2
- who:
Allen Wirfs-Brock
- bug_when:
2015-02-12 12:17:39 -0800
fixed in rev33