archives

« Bugzilla Issues Index

#3425 — 21.1.3.* String.prototype.{match, search}: Provide a default matcher/searcher

• bug_id: 3425
• creation_ts: 2014-12-08 13:58:00 -0800
• short_desc: 21.1.3.* String.prototype.{match, search}: Provide a default matcher/searcher
• delta_ts: 2014-12-23 20:23:30 -0800
• product: Draft for 6th Edition
• component: technical issue
• version: Rev 29: December 06, 2014 Draft
• rep_platform: All
• op_sys: All
• bug_status: RESOLVED
• resolution: FIXED
• priority: Normal
• bug_severity: normal
• everconfirmed: true
• reporter: Claude Pache
• assigned_to: Allen Wirfs-Brock
• cc: toshok

---

• commentid: 10961
• comment_count: 0
• who: Claude Pache
• bug_when: 2014-12-08 13:58:52 -0800

21.1.3.11 String.prototype.match
21.1.3.15 String.prototype.search

When the matcher/searcher found in step 4 is undefined, a new regexp is constructed in step 7a, but the original undefined matcher/searcher is incorrectly used in step 9.

A step 7b should be added:

Let matcher/searcher be the original value of RegExp.prototype[@@match/@@search].

---

• commentid: 10965
• comment_count: 1
• who: Allen Wirfs-Brock
• bug_when: 2014-12-08 14:51:55 -0800

fixed in rev30 editor's draft.
Several bugs besides the ones mentioned here. Also fixes in replace and split.

Here's what match now looks like

1. Let O be RequireObjectCoercible(this value).
2. Let S be ToString(O).
3. ReturnIfAbrupt(S).
4. Let matcher be GetMethod(regexp, @@match).
5. ReturnIfAbrupt(matcher).
6. If matcher is not undefined, then
a. Return Call(matcher, regexp, «S»).
7. Let rx be RegExpCreate(regexp, undefined) (see .2.3.3).
8. Return Invoke(rx, @@match, «S»).

---

• commentid: 11043
• comment_count: 2
• who: Allen Wirfs-Brock
• bug_when: 2014-12-15 13:37:37 -0800

*** Bug 3444 has been marked as a duplicate of this bug. ***

---

• commentid: 11191
• comment_count: 3
• who: Allen Wirfs-Brock
• bug_when: 2014-12-23 20:23:30 -0800

fixed in rev30