From dd0a36119bf4aed97b384c1a14ade39e2fdd35da Mon Sep 17 00:00:00 2001 From: "Alex Lam S.L" Date: Wed, 8 Nov 2017 02:10:46 +0800 Subject: [PATCH] fix `const` under `collapse_vars` without `unused` (#2454) fixes #2453 --- lib/compress.js | 3 ++- test/compress/collapse_vars.js | 22 ++++++++++++++++++++++ 2 files changed, 24 insertions(+), 1 deletion(-) diff --git a/lib/compress.js b/lib/compress.js index 8c634db2..8ebecd7f 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -1070,7 +1070,8 @@ merge(Compressor.prototype, { candidates.push(expr); } else if (expr instanceof AST_Sequence) { expr.expressions.forEach(extract_candidates); - } else if (expr instanceof AST_Definitions) { + } else if (expr instanceof AST_Definitions + && (compressor.option("unused") || !(expr instanceof AST_Const))) { expr.definitions.forEach(function(var_def) { if (var_def.value) candidates.push(var_def); }); diff --git a/test/compress/collapse_vars.js b/test/compress/collapse_vars.js index 4e7bd323..ee6a7366 100644 --- a/test/compress/collapse_vars.js +++ b/test/compress/collapse_vars.js @@ -3355,3 +3355,25 @@ issue_2437: { }(); } } + +issue_2453: { + options = { + collapse_vars: true, + toplevel: true, + } + input: { + function log(n) { + console.log(n); + } + const a = 42; + log(a); + } + expect: { + function log(n) { + console.log(n); + } + const a = 42; + log(a); + } + expect_stdout: "42" +}