avoid double counting within single-use functions (#2785)
authorAlex Lam S.L <alexlamsl@gmail.com>
Mon, 15 Jan 2018 08:42:15 +0000 (16:42 +0800)
committerGitHub <noreply@github.com>
Mon, 15 Jan 2018 08:42:15 +0000 (16:42 +0800)
fixes #2783

lib/compress.js
test/compress/functions.js

index 321a134..7e3503f 100644 (file)
@@ -5022,6 +5022,7 @@ merge(Compressor.prototype, {
             }
             if (single_use && fixed) {
                 if (fixed instanceof AST_Defun) {
+                    fixed._squeezed = true;
                     fixed = make_node(AST_Function, fixed, fixed);
                 }
                 var value;
index 222aa0c..2d55dd5 100644 (file)
@@ -1951,3 +1951,41 @@ issue_2737_2: {
     }
     expect_stdout: "PASS"
 }
+
+issue_2783: {
+    options = {
+        collapse_vars: true,
+        conditionals: true,
+        if_return: true,
+        inline: true,
+        reduce_vars: true,
+        unused: true,
+    }
+    input: {
+        (function() {
+            return g;
+            function f(a) {
+                var b = a.b;
+                if (b) return b;
+                return a;
+            }
+            function g(o, i) {
+                while (i--) {
+                    console.log(f(o));
+                }
+            }
+        })()({ b: "PASS" }, 1);
+    }
+    expect: {
+        (function() {
+            return function(o,i) {
+                while (i--) console.log(f(o));
+            };
+            function f(a) {
+                var b = a.b;
+                return b || a;
+            }
+        })()({ b: "PASS" },1);
+    }
+    expect_stdout: "PASS"
+}