From: Alex Lam S.L Date: Fri, 15 Dec 2017 18:16:35 +0000 (+0800) Subject: fix nested `inline` (#2602) X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=6c686ce59342c42b7fdcf54296e28c6c6517e6ab;p=UglifyJS.git fix nested `inline` (#2602) fixes #2601 --- diff --git a/lib/compress.js b/lib/compress.js index 0fac3259..8c17422d 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -3942,6 +3942,7 @@ merge(Compressor.prototype, { expressions.unshift(value || make_node(AST_Undefined, self)); } } else { + scope.var_names()[name.name] = true; decls.unshift(make_node(AST_VarDef, name, { name: name, value: null diff --git a/test/compress/functions.js b/test/compress/functions.js index a36509af..8e67bc32 100644 --- a/test/compress/functions.js +++ b/test/compress/functions.js @@ -772,3 +772,78 @@ issue_2476: { } expect_stdout: "465" } + +issue_2601_1: { + options = { + inline: true, + reduce_vars: true, + sequences: true, + side_effects: true, + unused: true, + } + input: { + var a = "FAIL"; + (function() { + function f(b) { + function g(b) { + b && b(); + } + g(); + (function() { + b && (a = "PASS"); + })(); + } + f("foo"); + })(); + console.log(a); + } + expect: { + var a = "FAIL"; + (function() { + b = "foo", + function(b) { + b && b(); + }(), + b && (a = "PASS"); + var b; + })(), + console.log(a); + } + expect_stdout: "PASS" +} + +issue_2601_2: { + rename = true + options = { + evaluate: true, + inline: true, + passes: 3, + reduce_vars: true, + sequences: true, + side_effects: true, + unused: true, + } + mangle = {} + input: { + var a = "FAIL"; + (function() { + function f(b) { + function g(b) { + b && b(); + } + g(); + (function() { + b && (a = "PASS"); + })(); + } + f("foo"); + })(); + console.log(a); + } + expect: { + var a = "FAIL"; + a = "PASS", + console.log(a); + } + expect_stdout: "PASS" +}