diff --git a/lib/compress.js b/lib/compress.js index 24129a05..d1ba2986 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -668,6 +668,7 @@ merge(Compressor.prototype, { function can_be_evicted_from_block(node) { return !( node instanceof AST_DefClass || + node instanceof AST_Defun || node instanceof AST_Let || node instanceof AST_Const ); @@ -1997,7 +1998,8 @@ merge(Compressor.prototype, { self.body = tighten_body(self.body, compressor); switch (self.body.length) { case 1: - if (can_be_evicted_from_block(self.body[0])) { + if (!compressor.has_directive("use strict") && compressor.parent() instanceof AST_If + || can_be_evicted_from_block(self.body[0])) { return self.body[0]; } break; diff --git a/test/compress/blocks.js b/test/compress/blocks.js index 8372adf2..971dd861 100644 --- a/test/compress/blocks.js +++ b/test/compress/blocks.js @@ -47,3 +47,142 @@ keep_some_blocks: { } else stuff(); } } + +issue_1664: { + input: { + var a = 1; + function f() { + if (undefined) a = 2; + { + function undefined() {} + undefined(); + } + } + f(); + console.log(a); + } + expect: { + var a = 1; + function f() { + if (undefined) a = 2; + { + function undefined() {} + undefined(); + } + } + f(); + console.log(a); + } + expect_stdout: "1" + node_version: ">=6" +} + +issue_1672_for: { + input: { + switch (function() { + return xxx; + }) { + case xxx: + for (; console.log("FAIL");) { + function xxx() {} + } + break; + } + } + expect: { + switch (function() { + return xxx; + }) { + case xxx: + for (; console.log("FAIL");) { + function xxx() {} + } + break; + } + } + expect_stdout: true + node_version: ">=6" +} + +issue_1672_for_strict: { + input: { + "use strict"; + switch (function() { + return xxx; + }) { + case xxx: + for (; console.log("FAIL");) { + function xxx() {} + } + break; + } + } + expect: { + "use strict"; + switch (function() { + return xxx; + }) { + case xxx: + for (; console.log("FAIL");) { + function xxx() {} + } + break; + } + } + expect_stdout: true + node_version: ">=6" +} + +issue_1672_if: { + input: { + switch (function() { + return xxx; + }) { + case xxx: + if (console.log("FAIL")) { + function xxx() {} + } + break; + } + } + expect: { + switch (function() { + return xxx; + }) { + case xxx: + if (console.log("FAIL")) function xxx() {} + break; + } + } + expect_stdout: true + node_version: ">=6" +} + +issue_1672_if_strict: { + input: { + "use strict"; + switch (function() { + return xxx; + }) { + case xxx: + if (console.log("FAIL")) { + function xxx() {} + } + break; + } + } + expect: { + "use strict"; + switch (function() { + return xxx; + }) { + case xxx: + if (console.log("FAIL")) { + function xxx() {} + } + break; + } + } + expect_stdout: true + node_version: ">=6" +}