From d1aa09c5c7af14bf5f17cc7ea2ab5d6be20e3220 Mon Sep 17 00:00:00 2001 From: "Alex Lam S.L" Date: Tue, 18 Apr 2017 01:44:23 +0800 Subject: [PATCH] fix `reduce_vars` on conditionals (#1822) --- lib/compress.js | 10 ++++++++++ test/compress/reduce_vars.js | 23 +++++++++++++++++++++++ 2 files changed, 33 insertions(+) diff --git a/lib/compress.js b/lib/compress.js index f49dd60e..0dfe2a3c 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -348,6 +348,16 @@ merge(Compressor.prototype, { pop(); return true; } + if (node instanceof AST_Conditional) { + node.condition.walk(tw); + push(); + node.consequent.walk(tw); + pop(); + push(); + node.alternative.walk(tw); + pop(); + return true; + } if (node instanceof AST_If) { node.condition.walk(tw); push(); diff --git a/test/compress/reduce_vars.js b/test/compress/reduce_vars.js index 94d37cb7..ad2c90bc 100644 --- a/test/compress/reduce_vars.js +++ b/test/compress/reduce_vars.js @@ -2240,3 +2240,26 @@ boolean_binary_assign: { } expect_stdout: "undefined" } + +cond_assign: { + options = { + evaluate: true, + reduce_vars: true, + unused: true, + } + input: { + !function() { + var a; + void 0 ? (a = 1) : 0; + console.log(a); + }(); + } + expect: { + !function() { + var a; + void 0 ? (a = 1) : 0; + console.log(a); + }(); + } + expect_stdout: "undefined" +} -- 2.34.1