fix corner case in unused (#5668)

alexlamsl · web-flow · commit eb22f0101eba · 2022-09-17T13:32:41.000+08:00
fixes #5663
diff --git a/lib/compress.js b/lib/compress.js
@@ -7937,7 +7937,14 @@ Compressor.prototype.compress = function(node) {
                         drop = false;
                         value = value.fixed_value();
                     }
-                    var values = value instanceof AST_Array && value.elements;
+                    var native, values;
+                    if (value instanceof AST_Array) {
+                        native = true;
+                        values = value.elements;
+                    } else {
+                        native = value && value.is_string(compressor);
+                        values = false;
+                    }
                     var elements = [], newValues = drop && [], pos = 0;
                     node.elements.forEach(function(element, index) {
                         value = values && values[index];
@@ -7988,8 +7995,9 @@ Compressor.prototype.compress = function(node) {
                         value = value.clone();
                         value.elements = newValues;
                     }
-                    if (!node.rest && (value instanceof AST_Array
-                        || value && value.is_string(compressor))) switch (elements.length) {
+                    if (!native) {
+                        elements.length = node.elements.length;
+                    } else if (!node.rest) switch (elements.length) {
                       case 0:
                         if (node === root) break;
                         if (drop) value = value.drop_side_effect_free(compressor);
diff --git a/test/compress/default-values.js b/test/compress/default-values.js
@@ -2713,7 +2713,7 @@ issue_5533_2_drop_fargs: {
         try {
             (function() {
                 for (;;) {
-                    var [ [] = [] ] = [];
+                    var [ [ , ] = [] ] = [];
                     throw "PASS";
                 }
             })();
diff --git a/test/compress/destructured.js b/test/compress/destructured.js
@@ -1306,7 +1306,7 @@ keep_reference: {
     }
     expect: {
         var a = [ {}, 42 ];
-        var [ b ] = a;
+        var b = a[0];
         console.log(a[0] === b ? "PASS" : "FAIL");
     }
     expect_stdout: "PASS"
diff --git a/test/compress/yields.js b/test/compress/yields.js
@@ -1735,3 +1735,37 @@ issue_5576: {
     ]
     node_version: ">=10"
 }
+
+issue_5663: {
+    options = {
+        toplevel: true,
+        unused: true,
+    }
+    input: {
+        var [ , a ] = function*() {
+            console.log("foo");
+            yield console.log("bar");
+            console.log("baz");
+            yield console.log("moo");
+            console.log("moz");
+            yield FAIL;
+        }();
+    }
+    expect: {
+        var [ , , ] = function*() {
+            console.log("foo");
+            yield console.log("bar");
+            console.log("baz");
+            yield console.log("moo");
+            console.log("moz");
+            yield FAIL;
+        }();
+    }
+    expect_stdout: [
+        "foo",
+        "bar",
+        "baz",
+        "moo",
+    ]
+    node_version: ">=6"
+}

Original file line number	Diff line number	Diff line change
`@@ -2713,7 +2713,7 @@ issue_5533_2_drop_fargs: {`
`2713`	`2713`	`try {`
`2714`	`2714`	`(function() {`
`2715`	`2715`	`for (;;) {`
`2716`		`- var [ [] = [] ] = [];`
	`2716`	`+ var [ [ , ] = [] ] = [];`
`2717`	`2717`	`throw "PASS";`
`2718`	`2718`	`}`
`2719`	`2719`	`})();`
Original file line number	Diff line number	Diff line change
`@@ -1306,7 +1306,7 @@ keep_reference: {`
`1306`	`1306`	`}`
`1307`	`1307`	`expect: {`
`1308`	`1308`	`var a = [ {}, 42 ];`
`1309`		`- var [ b ] = a;`
	`1309`	`+ var b = a[0];`
`1310`	`1310`	`console.log(a[0] === b ? "PASS" : "FAIL");`
`1311`	`1311`	`}`
`1312`	`1312`	`expect_stdout: "PASS"`