Do not change function arity while transforming. Closes #1331

terser · Feb 4, 2023 · 6268e5f · 6268e5f
1 parent 8dc5fc2
commit 6268e5f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/transform.js b/lib/transform.js
@@ -204,7 +204,7 @@ def_transform(AST_Destructuring, function(self, tw) {
 
 def_transform(AST_Lambda, function(self, tw) {
     if (self.name) self.name = self.name.transform(tw);
-    self.argnames = do_list(self.argnames, tw);
+    self.argnames = do_list(self.argnames, tw, /* allow_splicing */ false);
     if (self.body instanceof AST_Node) {
         self.body = self.body.transform(tw);
     } else {
@@ -214,7 +214,7 @@ def_transform(AST_Lambda, function(self, tw) {
 
 def_transform(AST_Call, function(self, tw) {
     self.expression = self.expression.transform(tw);
-    self.args = do_list(self.args, tw);
+    self.args = do_list(self.args, tw, /* allow_splicing */ false);
 });
 
 def_transform(AST_Sequence, function(self, tw) {

diff --git a/lib/utils/index.js b/lib/utils/index.js
@@ -98,11 +98,11 @@ function return_this() { return this; }
 function return_null() { return null; }
 
 var MAP = (function() {
-    function MAP(a, tw) {
+    function MAP(a, tw, allow_splicing = true) {
         // Loop but try not to build a new array
         for (let i = 0; i < a.length; ++i) {
             let item = a[i];
-            let ret = item.transform(tw, true);
+            let ret = item.transform(tw, allow_splicing);
 
             if (ret !== item) {
                 const a1 = a.slice(0, i);