diff options
author | Alex Lam S.L <alexlamsl@gmail.com> | 2017-08-29 01:10:04 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-08-29 01:10:04 +0800 |
commit | e5cf7972eab25581f9aa020bc8dbdd8bf25743cb (patch) | |
tree | 8f432a66a483ddbedfab83fcefbe82ed5f3d70bb | |
parent | f81ff10a9b9b3f65ad78bde5b2dbbc5739c70529 (diff) | |
download | tracifyjs-e5cf7972eab25581f9aa020bc8dbdd8bf25743cb.tar.gz tracifyjs-e5cf7972eab25581f9aa020bc8dbdd8bf25743cb.zip |
fix `unused` patching of `AST_For.init` blocks (#2289)
fixes #2288
-rw-r--r-- | lib/compress.js | 9 | ||||
-rw-r--r-- | test/compress/drop-unused.js | 22 |
2 files changed, 27 insertions, 4 deletions
diff --git a/lib/compress.js b/lib/compress.js index 7a16ba86..9121d0ce 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -2423,17 +2423,18 @@ merge(Compressor.prototype, { // We fix it at this stage by moving the `var` outside the `for`. if (node instanceof AST_For) { descend(node, this); + var block; if (node.init instanceof AST_BlockStatement) { - var block = node.init; + block = node.init; node.init = block.body.pop(); block.body.push(node); - return in_list ? MAP.splice(block.body) : block; - } else if (node.init instanceof AST_SimpleStatement) { + } + if (node.init instanceof AST_SimpleStatement) { node.init = node.init.body; } else if (is_empty(node.init)) { node.init = null; } - return node; + return !block ? node : in_list ? MAP.splice(block.body) : block; } if (node instanceof AST_LabeledStatement && node.body instanceof AST_For) { descend(node, this); diff --git a/test/compress/drop-unused.js b/test/compress/drop-unused.js index 34d47d0d..c9048540 100644 --- a/test/compress/drop-unused.js +++ b/test/compress/drop-unused.js @@ -1239,3 +1239,25 @@ issue_2226_3: { } expect_stdout: "3" } + +issue_2288: { + options = { + unused: true, + } + beautify = { + beautify: true, + } + input: { + function foo(o) { + for (var j = o.a, i = 0; i < 0; i++); + for (var i = 0; i < 0; i++); + } + } + expect: { + function foo(o) { + o.a; + for (i = 0; i < 0; i++); + for (var i = 0; i < 0; i++); + } + } +} |