aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--lib/compress.js45
-rw-r--r--test/compress/merge_vars.js81
2 files changed, 118 insertions, 8 deletions
diff --git a/lib/compress.js b/lib/compress.js
index e4479ca8..c6e27e06 100644
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -4468,6 +4468,35 @@ merge(Compressor.prototype, {
});
return true;
}
+ if (node instanceof AST_Continue) {
+ var target = tw.loopcontrol_target(node);
+ if (!(target instanceof AST_Do)) return true;
+ var stack = [];
+ while (!HOP(segment, "block") || segment.block !== target) {
+ stack.push(segment);
+ pop();
+ }
+ segment.loop = "c";
+ push();
+ while (stack.length) {
+ var seg = stack.pop();
+ push();
+ if (HOP(seg, "block")) segment.block = seg.block;
+ if (HOP(seg, "loop")) segment.loop = seg.loop;
+ }
+ return true;
+ }
+ if (node instanceof AST_Do) {
+ push();
+ segment.block = node;
+ segment.loop = true;
+ var save = segment;
+ node.body.walk(tw);
+ if (segment.loop == "c") segment = save;
+ node.condition.walk(tw);
+ pop();
+ return true;
+ }
if (node instanceof AST_For) {
if (node.init) node.init.walk(tw);
push();
@@ -4501,14 +4530,6 @@ merge(Compressor.prototype, {
}
return true;
}
- if (node instanceof AST_IterationStatement) {
- push();
- segment.block = node;
- segment.loop = true;
- descend();
- pop();
- return true;
- }
if (node instanceof AST_LabeledStatement) {
push();
segment.block = node;
@@ -4601,6 +4622,14 @@ merge(Compressor.prototype, {
}
return true;
}
+ if (node instanceof AST_While) {
+ push();
+ segment.block = node;
+ segment.loop = true;
+ descend();
+ pop();
+ return true;
+ }
});
tw.directives = Object.create(compressor.directives);
self.walk(tw);
diff --git a/test/compress/merge_vars.js b/test/compress/merge_vars.js
index 28a0135d..6ada9297 100644
--- a/test/compress/merge_vars.js
+++ b/test/compress/merge_vars.js
@@ -3011,3 +3011,84 @@ issue_4168_use_strict: {
}
expect_stdout: "PASS true 42"
}
+
+issue_4237_1: {
+ options = {
+ merge_vars: true,
+ }
+ input: {
+ console.log(function(a) {
+ do {
+ var b = a++;
+ if (b)
+ return "FAIL";
+ continue;
+ var c = 42;
+ } while ("undefined" != typeof c);
+ return "PASS";
+ }(0));
+ }
+ expect: {
+ console.log(function(a) {
+ do {
+ var b = a++;
+ if (b)
+ return "FAIL";
+ continue;
+ var c = 42;
+ } while ("undefined" != typeof c);
+ return "PASS";
+ }(0));
+ }
+ expect_stdout: "PASS"
+}
+
+issue_4237_2: {
+ options = {
+ dead_code: true,
+ evaluate: true,
+ loops: true,
+ merge_vars: true,
+ switches: true,
+ }
+ input: {
+ console.log(function(a) {
+ do {
+ switch (0) {
+ case 0:
+ var b = a++;
+ default:
+ while (b)
+ return "FAIL";
+ }
+ try {
+ var c = 0;
+ } finally {
+ continue;
+ }
+ var d = 0;
+ } while ("undefined" != typeof d);
+ return "PASS";
+ }(0));
+ }
+ expect: {
+ console.log(function(a) {
+ do {
+ switch (0) {
+ case 0:
+ var b = a++;
+ if (b)
+ return "FAIL";
+ }
+ try {
+ var c = 0;
+ } finally {
+ continue;
+ }
+ var d = 0;
+ } while ("undefined" != typeof d);
+ return "PASS";
+ }(0));
+ }
+ expect_stdout: "PASS"
+}