JS: support ellipsis operator anywhere in arguments, fixes #373

tdewolff · Feb 6, 2021 · 29261c3 · 29261c3
1 parent 512135e
commit 29261c3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/js/js.go b/js/js.go
@@ -477,14 +477,10 @@ func (m *jsMinifier) minifyArguments(args js.Args) {
 		if i != 0 {
 			m.write(commaBytes)
 		}
-		m.minifyExpr(item, js.OpAssign)
-	}
-	if args.Rest != nil {
-		if len(args.List) != 0 {
-			m.write(commaBytes)
+		if item.Rest {
+			m.write(ellipsisBytes)
 		}
-		m.write(ellipsisBytes)
-		m.minifyExpr(args.Rest, js.OpAssign)
+		m.minifyExpr(item.Value, js.OpAssign)
 	}
 	m.write(closeParenBytes)
 }

diff --git a/js/js_test.go b/js/js_test.go
@@ -505,6 +505,7 @@ func TestJS(t *testing.T) {
 		{`(1,2,a|3)&&b`, `1,2,a|3&&b`},
 		{`(a,b)?c:b`, `a,b&&c`},
 		{`(a,b)?c:d`, `a,b?c:d`},
+		{`f(...a,...b)`, `f(...a,...b)`},
 
 		// expressions
 		//{`a=a+5`, `a+=5`},