fix corner case in `functions` & `reduce_vars` (#4452)

fixes #4451

diff --git a/lib/compress.js b/lib/compress.js
index a661577..33aa5b9 100644 (file)
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -933,10 +933,10 @@ merge(Compressor.prototype, {
                         def.fixed = false;
                     }
                 }, true);
-            } else if (init instanceof AST_SymbolRef && !init.is_immutable()) {
+            } else if (init instanceof AST_SymbolRef) {
                 var def = init.definition();
                 def.assignments++;
-                def.fixed = false;
+                if (!init.is_immutable()) def.fixed = false;
             }
             this.body.walk(tw);
             pop(tw);

diff --git a/test/compress/functions.js b/test/compress/functions.js
index 837d0a0..066bc3c 100644 (file)
--- a/test/compress/functions.js
+++ b/test/compress/functions.js
@@ -5134,8 +5134,8 @@ issue_4259: {
         console.log(typeof a);
     }
     expect: {
-        function a() {
-            for (a in a);
+        var a = function b() {
+            for (b in b);
         }
         a();
         console.log(typeof a);
@@ -5225,3 +5225,27 @@ trailing_comma: {
     expect_exact: 'new function(a,b){console.log(b,a)}(42,"PASS");'
     expect_stdout: "PASS 42"
 }
+
+issue_4451: {
+    options = {
+        functions: true,
+        reduce_vars: true,
+        toplevel: true,
+        unused: true,
+    }
+    input: {
+        var a = function f() {
+            for (f in "foo")
+                return f;
+        };
+        while (console.log(typeof a()));
+    }
+    expect: {
+        var a = function f() {
+            for (f in "foo")
+                return f;
+        };
+        while (console.log(typeof a()));
+    }
+    expect_stdout: "function"
+}