fix corner case in `collapse_vars` (#4309)

fixes #4308

diff --git a/lib/compress.js b/lib/compress.js
index 9048880..601f39f 100644 (file)
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -1825,7 +1825,11 @@ merge(Compressor.prototype, {
                             name: sym,
                             value: arg
                         }));
-                        if (sym instanceof AST_Destructured) continue;
+                        if (sym instanceof AST_Destructured) {
+                            if (!sym.match_symbol(return_false)) continue;
+                            candidates.length = 0;
+                            break;
+                        }
                         if (sym.name in names) continue;
                         names[sym.name] = true;
                         if (!arg) {

diff --git a/test/compress/destructured.js b/test/compress/destructured.js
index bbe62a7..f8e8090 100644 (file)
--- a/test/compress/destructured.js
+++ b/test/compress/destructured.js
@@ -1591,3 +1591,28 @@ issue_4301: {
     expect_stdout: true
     node_version: ">=6"
 }
+
+issue_4308: {
+    options = {
+        collapse_vars: true,
+        unused: true,
+    }
+    input: {
+        var a = "PASS";
+        console.log(function({
+            [a = "FAIL"]: b
+        }, c) {
+            return c;
+        }(0, a));
+    }
+    expect: {
+        var a = "PASS";
+        console.log(function({
+            [a = "FAIL"]: b
+        }, c) {
+            return c;
+        }(0, a));
+    }
+    expect_stdout: "PASS"
+    node_version: ">=6"
+}