« Bugzilla Issues Index
#4066 — 20.3.4.27: algorithm simplification
- bug_id:
4066
- creation_ts:
2015-02-21 10:22:00 -0800
- short_desc:
20.3.4.27: algorithm simplification
- delta_ts:
2015-03-17 16:57:03 -0700
- product:
Draft for 6th Edition
- component:
editorial issue
- version:
Rev 35: March 4, 2015 Release Candidate 2
- rep_platform:
All
- op_sys:
All
- bug_status:
RESOLVED
- resolution:
FIXED
- priority:
Normal
- bug_severity:
enhancement
- everconfirmed:
true
- reporter:
Michael Dyck
- assigned_to:
Allen Wirfs-Brock
- commentid:
13240
- comment_count:
0
- who:
Michael Dyck
- bug_when:
2015-02-21 10:22:52 -0800
In 20.3.4.27 "Date.prototype.setTime ( time )",
steps 1 and 2 say:
1. Let O be the this value.
2. If Type(O) is not Object or O does not have a [[DateValue]] internal slot,
throw a TypeError exeption.
In the specification of other Date.prototype functions, these checks are accomplished in thisTimeValue(), which is implicitly invoked by referring to "this time value". So for consistency, replace the above steps with:
1. Let t be this time value.
2. ReturnIfAbrupt(t).
(or just "ReturnIfAbrupt(this time value)", since we don't need the value of t).
And then in step 6, replace /O/ with "this Date object".
- commentid:
13278
- comment_count:
1
- who:
Allen Wirfs-Brock
- bug_when:
2015-02-24 10:52:47 -0800
fixed in rev35 editor's draft
- commentid:
13468
- comment_count:
2
- who:
Allen Wirfs-Brock
- bug_when:
2015-03-04 18:58:12 -0800
fixed in rev35
- commentid:
13584
- comment_count:
3
- who:
Michael Dyck
- bug_when:
2015-03-05 12:17:39 -0800
Almost. You still need to change "the this value" to "this time value",
otherwise the more detailed checks no longer happen.
- commentid:
13699
- comment_count:
4
- who:
Allen Wirfs-Brock
- bug_when:
2015-03-11 14:03:28 -0700
fixed in rev36 editor's draft
- commentid:
13801
- comment_count:
5
- who:
Allen Wirfs-Brock
- bug_when:
2015-03-17 16:57:03 -0700
in rev36