archives

« Bugzilla Issues Index

#3236 — 12.3.3.1.1 EvaluateNew, 12.3.5.2 Evaluation: Replace prose with call to ArgumentListEvaluation

• bug_id: 3236
• creation_ts: 2014-09-19 08:49:00 -0700
• short_desc: 12.3.3.1.1 EvaluateNew, 12.3.5.2 Evaluation: Replace prose with call to ArgumentListEvaluation
• delta_ts: 2014-10-14 15:18:04 -0700
• product: Draft for 6th Edition
• component: editorial issue
• version: Rev 27: August 24, 2014 Draft
• 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: 10203
• comment_count: 0
• who: André Bargull
• bug_when: 2014-09-19 08:49:57 -0700

12.3.3.1.1 Runtime Semantics: EvaluateNew(thisCall, constructProduction, arguments)
12.3.5.2 Runtime Semantics: Evaluation
MemberExpression : new super Arguments

The current text:

> Let argList be the result of evaluating Arguments, producing a List of argument values (12.3.6).

Can be replaced with:

> Let argList be ArgumentListEvaluation(arguments).

And in 12.3.5.2 with:

> Let argList be ArgumentListEvaluation(Arguments).



Also: Typo in 12.3.3.1.1, step 8a "Arguments" -> "arguments"


Also: Can EvaluateNew be restructured, so that the duplicated text for "MemberExpression : new super Arguments" and "NewExpression : new super" gets away?

---

• commentid: 10216
• comment_count: 1
• who: Allen Wirfs-Brock
• bug_when: 2014-09-21 10:16:05 -0700

fixed in rev28 editor's draft.

the bugs, that is.

I don't see an obviously way to further simply as the text for "MemberExpression : new super Arguments" and "NewExpression : new super" is just setup for EvaluateNew.

---

• commentid: 10234
• comment_count: 2
• who: André Bargull
• bug_when: 2014-09-21 13:03:25 -0700

(In reply to Allen Wirfs-Brock from comment #1)
> I don't see an obviously way to further simply as the text for
> "MemberExpression : new super Arguments" and "NewExpression : new super" is
> just setup for EvaluateNew.

"MemberExpression : new super Arguments" and "NewExpression : new super" don't use EvaluateNew in rev27, maybe this was already changed for rev28? Basically I was just asking if it makes sense to restructure EvaluateNew so it works similar to EvaluateCall. Something like: https://gist.github.com/anba/513f97a86f8a44182606

---

• commentid: 10494
• comment_count: 3
• who: Allen Wirfs-Brock
• bug_when: 2014-10-14 15:18:04 -0700

fixed in rev28