cerxes/babel
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

add support for trailing commas in arrow function parameter lists - fixes #1841

Browse Source
This commit is contained in:
Sebastian McKenzie 2015-06-26 00:37:33 +01:00
parent 030d5f7c6b
commit 84c773a7ca
1 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/expression.js
View File

@ -432,9 +432,18 @@ pp.parseParenAndDistinguishExpression = function(start, isAsync, canBeArrow) {
} }
let innerStart = this.markPosition(), exprList = [], first = true let innerStart = this.markPosition(), exprList = [], first = true
let refShorthandDefaultPos = {start: 0}, spreadStart, innerParenStart let refShorthandDefaultPos = {start: 0}, spreadStart, innerParenStart, optionalCommaStart
while (this.type !== tt.parenR) { while (this.type !== tt.parenR) {
first ? first = false : this.expect(tt.comma) if (first) {
first = false;
} else {
this.expect(tt.comma)
if (this.type === tt.parenR && this.options.features["es7.trailingFunctionCommas"]) {
optionalCommaStart = this.start
break
}
}
if (this.type === tt.ellipsis) { if (this.type === tt.ellipsis) {
let spreadNodeStart = this.markPosition() let spreadNodeStart = this.markPosition()
spreadStart = this.start spreadStart = this.start
@ -462,6 +471,7 @@ pp.parseParenAndDistinguishExpression = function(start, isAsync, canBeArrow) {
this.unexpected(this.lastTokStart) this.unexpected(this.lastTokStart)
} }
} }
if (optionalCommaStart) this.unexpected(optionalCommaStart)
if (spreadStart) this.unexpected(spreadStart) if (spreadStart) this.unexpected(spreadStart)
if (refShorthandDefaultPos.start) this.unexpected(refShorthandDefaultPos.start) if (refShorthandDefaultPos.start) this.unexpected(refShorthandDefaultPos.start)