« Bugzilla Issues Index
#4055 — 9.2.2.1 PrepareForOrdinaryCall: Unnecessary ReturnIfAbrupt in step 8
- bug_id:
4055
- creation_ts:
2015-02-21 08:34:00 -0800
- short_desc:
9.2.2.1 PrepareForOrdinaryCall: Unnecessary ReturnIfAbrupt in step 8
- delta_ts:
2015-03-04 18:58:18 -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
- blocked:
4076
- everconfirmed:
true
- reporter:
André Bargull
- assigned_to:
Allen Wirfs-Brock
- cc:
claude.pache
- commentid:
13229
- comment_count:
0
- who:
André Bargull
- bug_when:
2015-02-21 08:34:17 -0800
9.2.2.1 PrepareForOrdinaryCall( F, newTarget )
Steps 8-9 can be removed, because PrepareForOrdinaryCall is no longer fallible.
- commentid:
13362
- comment_count:
1
- who:
Allen Wirfs-Brock
- bug_when:
2015-02-25 15:48:29 -0800
This isn't explicitly dealt with in NewFunctionEnvironment but what if the new Environment can't be created because of low memory or stack overflow?
- commentid:
13383
- comment_count:
2
- who:
André Bargull
- bug_when:
2015-02-26 02:52:53 -0800
OOM or stack overflow should be ignored by the specification, otherwise a whole lot more operations need ReturnIfAbrupt calls. For example, string concatenation or recursive abstract operation calls etc.
- commentid:
13391
- comment_count:
3
- who:
Allen Wirfs-Brock
- bug_when:
2015-02-26 10:00:44 -0800
fixed in rev35 editor's draft
yes, that's generally how it's done except for AllocateArrayBuffer which does need an explicit can't allocate exception.
- commentid:
13521
- comment_count:
4
- who:
Allen Wirfs-Brock
- bug_when:
2015-03-04 18:58:18 -0800
fixed in rev35