From: Alex Lam S.L Date: Mon, 15 Jan 2018 08:42:15 +0000 (+0800) Subject: avoid double counting within single-use functions (#2785) X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=cbbe6fad60dd073c63575f591d631efa94bbceaf;p=UglifyJS.git avoid double counting within single-use functions (#2785) fixes #2783 --- diff --git a/lib/compress.js b/lib/compress.js index 321a1340..7e3503fe 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -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; diff --git a/test/compress/functions.js b/test/compress/functions.js index 222aa0ca..2d55dd52 100644 --- a/test/compress/functions.js +++ b/test/compress/functions.js @@ -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" +}