VOOZH about

URL: https://bugzilla.mozilla.org/show_bug.cgi?id=1096378

⇱ 1096378 - Don't allow duplicate parameter names in concise method definitions

Closed Bug 1096378 Opened 11 years ago Closed 11 years ago

Don't allow duplicate parameter names in concise method definitions

Don't allow duplicate parameter names in concise method definitions
Core
JavaScript Engine
Trunk
All
All
defect
Not set
normal
Points:
---
RESOLVED FIXED
RESOLVED
FIXED
mozilla38
Iteration:
---
a11y-review
Accessibility Severity
Performance Impact
Size Estimate
Webcompat Priority
Webcompat Score
Tracking Status
firefox38 --- fixed
Tracking Status
firefox38
relnote-firefox
thunderbird_esr115
thunderbird_esr140
thunderbird_esr153
firefox-esr115
firefox-esr140
firefox-esr153
firefox152
firefox153
firefox154
---
QA Whiteboard:
---
Has STR:
---
Change Request:
---
Signature:
None
This bug is publicly visible. 

 












 
 

 

 
 
 
 
1.61 KB,
 patch 

 		
 
 
jorendorff

:
 
 review+
 

 
 Details
 | Diff |
 Splinter Review
 





 
















Test case:
---
({m(a, a){}})
---

Expected: SyntaxError is thrown
Actual: No SyntaxError

Concise method definition parameters use the 'StrictFormalParameters' grammar production [1] which disallows duplicate parameter names [2].

[1] https://people.mozilla.org/~jorendorff/es6-draft.html#sec-method-definitions
[2] https://people.mozilla.org/~jorendorff/es6-draft.html#sec-function-definitions-static-semantics-early-errors

 Attachment #8553309 -
 Flags: review?(jorendorff) → review+
Assignee: nobody → arai_a
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla38

 You need to log in
 before you can comment on or make changes to this bug.