archives

« Bugzilla Issues Index

#3219 — Remove duplicate check for constructor

• bug_id: 3219
• creation_ts: 2014-09-10 16:02:00 -0700
• short_desc: Remove duplicate check for constructor
• delta_ts: 2015-07-10 08:34:21 -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: WONTFIX
• priority: Normal
• bug_severity: normal
• everconfirmed: true
• reporter: Erik Arvidsson
• assigned_to: Allen Wirfs-Brock
• cc: erik.arvidsson

---

• commentid: 10178
• comment_count: 0
• who: Erik Arvidsson
• bug_when: 2014-09-10 16:02:27 -0700

We have removed the duplicate property check in ES6 but we still check for duplicate "constructor" property:



14.5.1 Static Semantics: Early Errors

ClassBody : ClassElementList

It is a Syntax Error if PrototypePropertyNameList of ClassElementList contains more than one occurrence of "constructor".



Maybe we should remove this restriction too in the name of consistency?

---

• commentid: 10179
• comment_count: 1
• who: Allen Wirfs-Brock
• bug_when: 2014-09-10 16:32:19 -0700

I don't think, just like __proto__ in object literals, constructor in class definitions is a special form and not a regular property definition and semantically is quite different from
[expressionThatEvaluatesToConstrutor] () {}

If I was going to change anything, I'd make sure that trying to defining such a method was a runtime error.