« Bugzilla Issues Index
#4047 — 8.1.1.5.1 GetBindingValue: Wrong test in step 3.b
- bug_id:
4047
- creation_ts:
2015-02-21 08:32:00 -0800
- short_desc:
8.1.1.5.1 GetBindingValue: Wrong test in step 3.b
- delta_ts:
2015-03-04 18:58:14 -0800
- product:
Draft for 6th Edition
- component:
technical issue
- version:
Rev 34: February 20, 2015 Release Candidate 1
- 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:
13221
- comment_count:
0
- who:
André Bargull
- bug_when:
2015-02-21 08:32:54 -0800
8.1.1.5.1 GetBindingValue(N,S)
Steps 3.b-c should read:
> Let targetEnv be M.[[Environment]].
> If targetEnv is undefined, throw a ReferenceError exception.
> Let targetER be targetEnv’s environment record.
- commentid:
13335
- comment_count:
1
- who:
Allen Wirfs-Brock
- bug_when:
2015-02-24 16:31:03 -0800
fixed in rev35 editor's draft
I'm pretty sure (but not certain) that this condition never occurs. I considered an assert, but the uncertainty makes the throw a better choice.
- commentid:
13404
- comment_count:
2
- who:
André Bargull
- bug_when:
2015-02-26 14:46:49 -0800
(In reply to Allen Wirfs-Brock from comment #1)
> I'm pretty sure (but not certain) that this condition never occurs. I
> considered an assert, but the uncertainty makes the throw a better choice.
It shouldn't be possible with the built-in Source Text Module Record, but a Module Record implementation from a different specification may have other semantics. So I agree with saying that `throw` is the better choice.
- commentid:
13490
- comment_count:
3
- who:
Allen Wirfs-Brock
- bug_when:
2015-03-04 18:58:14 -0800
fixed in rev35