« Bugzilla Issues Index
#20 — String.proto.match, unnecessary [[Class]] test
- bug_id:
20
- creation_ts:
2011-02-14 15:42:00 -0800
- short_desc:
String.proto.match, unnecessary [[Class]] test
- delta_ts:
2013-03-08 14:44:32 -0800
- product:
Draft for 6th Edition
- component:
restructure
- version:
Initial draft July 12, 2011
- rep_platform:
All
- op_sys:
All
- bug_status:
RESOLVED
- resolution:
FIXED
- priority:
Low
- bug_severity:
minor
- everconfirmed:
true
- reporter:
Allen Wirfs-Brock
- assigned_to:
Allen Wirfs-Brock
- commentid:
36
- comment_count:
0
- who:
Allen Wirfs-Brock
- bug_when:
2011-02-14 15:42:52 -0800
Step 3&4 could be simplified to:
3 let rx be the value returned as if by calling the standard built-in RegExp constructor as a function with regexp as the argument.
This eliminates as [[class]] test and redundant specification language.
- commentid:
345
- comment_count:
1
- who:
Allen Wirfs-Brock
- bug_when:
2011-07-20 16:26:43 -0700
However, the current form may more cleary expression that if the argument is already a regexp it is used without conversion.
In any case, the use of [[Class]] has already been eliminate.
- commentid:
3301
- comment_count:
2
- who:
Allen Wirfs-Brock
- bug_when:
2013-03-07 14:44:18 -0800
fixed in rev 14 editor's draft
- commentid:
3393
- comment_count:
3
- who:
Allen Wirfs-Brock
- bug_when:
2013-03-08 14:44:32 -0800
in Rev 14 draft