From: Alex Lam S.L <alexlamsl@gmail.com>
Date: Fri, 28 Aug 2020 18:42:17 +0000 (+0100)
Subject: fix corner case in `loops` & `unused` (#4083)
X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=f886b3fb2bad77ca7158057b48add16050ceca2a;p=UglifyJS.git

fix corner case in `loops` & `unused` (#4083)

fixes #4082
---

diff --git a/lib/compress.js b/lib/compress.js
index 90dd0f27..bab390cf 100644
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -4800,7 +4800,7 @@ merge(Compressor.prototype, {
                 return true;
             }
             if (node instanceof AST_ForIn) {
-                if (!compressor.option("loops")) return;
+                if (!drop_vars || !compressor.option("loops")) return;
                 if (!is_empty(node.body)) return;
                 if (node.init.has_side_effects(compressor)) return;
                 node.object.walk(tw);
diff --git a/test/compress/loops.js b/test/compress/loops.js
index 23d46de1..e24b3712 100644
--- a/test/compress/loops.js
+++ b/test/compress/loops.js
@@ -985,3 +985,27 @@ issue_4075: {
     }
     expect_stdout: "PASS"
 }
+
+issue_4082: {
+    options = {
+        keep_fargs: "strict",
+        loops: true,
+        unused: true,
+    }
+    input: {
+        var a = "PASS";
+        (function(a) {
+            for (a in "foo")
+                var b;
+        })();
+        console.log(a);
+    }
+    expect: {
+        var a = "PASS";
+        (function(a) {
+            for (a in "foo");
+        })();
+        console.log(a);
+    }
+    expect_stdout: "PASS"
+}