archives

« Bugzilla Issues Index

#560 — 15.1.2.1: more execution context stuff

• bug_id: 560
• creation_ts: 2012-07-15 11:55:00 -0700
• short_desc: 15.1.2.1: more execution context stuff
• delta_ts: 2014-10-11 17:24:25 -0700
• product: Draft for 6th Edition
• component: editorial issue
• version: Rev 9: July 8, 2012 Draft
• rep_platform: All
• op_sys: All
• bug_status: RESOLVED
• resolution: FIXED
• priority: Normal
• bug_severity: normal
• everconfirmed: true
• reporter: Michael Dyck
• assigned_to: Allen Wirfs-Brock

---

• commentid: 1345
• comment_count: 0
• who: Michael Dyck
• bug_when: 2012-07-15 11:55:18 -0700

In 15.1.2.1 "eval(x)",
steps 5.b and 6.b say:
"Set the LexicalEnvironment to ..."

I think this should be:
"Set the LexicalEnvironment of evalCtx to ..."

Similarly for "VariableEnvironment" in 5.c and 6.c.

However, in 5.a,
"Set the ThisBinding to the same value as
the ThisBinding of the calling execution context."
is odd, because 10.4 doesn't say that an execution context has a ThisBinding.

---

• commentid: 1469
• comment_count: 1
• who: Allen Wirfs-Brock
• bug_when: 2012-08-13 19:01:07 -0700

this isjust confirming the note in 15.1.2.1 that is a reminder that the refactoring of eval needs to be finished.

Also, consider putting detection of direct eval into the call operator

---

• commentid: 8502
• comment_count: 2
• who: Michael Dyck
• bug_when: 2014-05-15 11:12:48 -0700

I believe all of my original points were addressed in rev11.

Don't know about the points in comment 1.