« Bugzilla Issues Index
#1373 — 15.5.4.11: "regexp" + "capture" + "replString" not defined
- bug_id:
1373
- creation_ts:
2013-03-21 09:54:00 -0700
- short_desc:
15.5.4.11: "regexp" + "capture" + "replString" not defined
- delta_ts:
2013-05-14 18:13:34 -0700
- product:
Draft for 6th Edition
- component:
editorial issue
- version:
Rev 14: March 8, 2013 Draft
- rep_platform:
All
- op_sys:
All
- bug_status:
RESOLVED
- resolution:
FIXED
- priority:
Normal
- bug_severity:
normal
- everconfirmed:
true
- reporter:
Michael Dyck
- assigned_to:
Allen Wirfs-Brock
- commentid:
3523
- comment_count:
0
- who:
Michael Dyck
- bug_when:
2013-03-21 09:54:45 -0700
In 15.5.4.11 "String.prototype.replace (searchValue, replaceValue)",
step 4.b says:
Return the result of Invoke(regexp, "replace", args).
but 'regexp' is not defined.
Change it to 'searchValue' ?
- commentid:
3524
- comment_count:
1
- who:
Michael Dyck
- bug_when:
2013-03-21 10:36:31 -0700
And step 10.b says:
Let replStr be the result of the abstract operation
GetReplaceSubstitution(matched, string, pos, capture).
but 'capture' is not defined.
Change to 'captures'.
(Alternatively, in step 10.a, you could change 'captures' to 'capture', since that would match the name of the corresponding parameter of GetReplaceSubstitution. However, that parameter should probably be changed to 'captures' too, to match usage elsewhere, and just because the plural makes more sense.)
- commentid:
3525
- comment_count:
2
- who:
Michael Dyck
- bug_when:
2013-03-21 10:45:44 -0700
And step 12 refers to 'replString', which is not defined.
Change to just 'replStr'.
- commentid:
3743
- comment_count:
3
- who:
Allen Wirfs-Brock
- bug_when:
2013-05-12 13:40:55 -0700
fixed in rev15 editor's draft.
- commentid:
3922
- comment_count:
4
- who:
Allen Wirfs-Brock
- bug_when:
2013-05-14 18:13:34 -0700
resolved in rev 15, May 14, 2013 draft