aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlex Lam S.L <alexlamsl@gmail.com>2017-04-02 17:07:20 +0800
committerGitHub <noreply@github.com>2017-04-02 17:07:20 +0800
commit9469c03ac976b3cfc1e52abf034ce76f1d6aca59 (patch)
tree05d5d5df8658191e2da841f01142e9279f73224d
parentd57527697fba37bfad50ca0283326a458cdea031 (diff)
downloadtracifyjs-9469c03ac976b3cfc1e52abf034ce76f1d6aca59.tar.gz
tracifyjs-9469c03ac976b3cfc1e52abf034ce76f1d6aca59.zip
fix corner case in `switch` (#1765)
-rw-r--r--lib/compress.js7
-rw-r--r--test/compress/issue-1750.js28
2 files changed, 31 insertions, 4 deletions
diff --git a/lib/compress.js b/lib/compress.js
index 763490a7..79e7d4d2 100644
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -2557,7 +2557,7 @@ merge(Compressor.prototype, {
if (!default_branch) {
default_branch = branch;
} else {
- eliminate_branch(branch);
+ eliminate_branch(branch, body[body.length - 1]);
}
} else if (value !== self.expression) {
var exp = branch.expression.evaluate(compressor);
@@ -2570,7 +2570,7 @@ merge(Compressor.prototype, {
default_branch = null;
}
} else if (exp !== branch.expression) {
- eliminate_branch(branch);
+ eliminate_branch(branch, body[body.length - 1]);
continue;
}
}
@@ -2583,7 +2583,7 @@ merge(Compressor.prototype, {
}
body.push(branch);
}
- while (i < len) eliminate_branch(self.body[i++]);
+ while (i < len) eliminate_branch(self.body[i++], body[body.length - 1]);
if (body.length > 0) {
body[0].body = decl.concat(body[0].body);
}
@@ -2626,7 +2626,6 @@ merge(Compressor.prototype, {
return self;
function eliminate_branch(branch, prev) {
- if (!prev) prev = body[body.length - 1];
if (prev && !aborts(prev)) {
prev.body = prev.body.concat(branch.body);
} else {
diff --git a/test/compress/issue-1750.js b/test/compress/issue-1750.js
index c1448afe..d18bc49f 100644
--- a/test/compress/issue-1750.js
+++ b/test/compress/issue-1750.js
@@ -24,3 +24,31 @@ case_1: {
}
expect_stdout: "0 2"
}
+
+case_2: {
+ options = {
+ dead_code: true,
+ evaluate: true,
+ switches: true,
+ }
+ input: {
+ var a = 0, b = 1;
+ switch (0) {
+ default:
+ b = 2;
+ case a:
+ a = 3;
+ case 0:
+ }
+ console.log(a, b);
+ }
+ expect: {
+ var a = 0, b = 1;
+ switch (0) {
+ case a:
+ a = 3;
+ }
+ console.log(a, b);
+ }
+ expect_stdout: "3 1"
+}