From 102f994b9d09b3ed0447f02a894fd88fcc903171 Mon Sep 17 00:00:00 2001 From: "Alex Lam S.L" Date: Thu, 1 Feb 2018 15:09:53 +0800 Subject: [PATCH] account for declaration assignment in `collapse_vars` (#2859) fixes #2858 --- lib/compress.js | 1 + test/compress/collapse_vars.js | 34 ++++++++++++++++++++++++++++++++++ 2 files changed, 35 insertions(+) diff --git a/lib/compress.js b/lib/compress.js index db6ee8be..9a3446c2 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -990,6 +990,7 @@ merge(Compressor.prototype, { || node instanceof AST_Debugger || node instanceof AST_IterationStatement && !(node instanceof AST_For) || node instanceof AST_Try + || node instanceof AST_VarDef && node.value && side_effects && !references_in_scope(node.name.definition()) || node instanceof AST_With || parent instanceof AST_For && node !== parent.init || (side_effects || !replace_all) diff --git a/test/compress/collapse_vars.js b/test/compress/collapse_vars.js index b3617816..f56face6 100644 --- a/test/compress/collapse_vars.js +++ b/test/compress/collapse_vars.js @@ -4190,3 +4190,37 @@ return_3: { } expect_stdout: "0" } + +issue_2858: { + options = { + collapse_vars: true, + unused: true, + } + input: { + var b; + (function() { + function f() { + a++; + } + f(); + var c = f(); + var a = void 0; + c || (b = a); + })(); + console.log(b); + } + expect: { + var b; + (function() { + function f() { + a++; + } + f(); + var c = f(); + var a = void 0; + c || (b = a); + })(); + console.log(b); + } + expect_stdout: "undefined" +} -- 2.34.1