JoBoEVD/UglifyJS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix corner case in merge_vars (#4158)

Browse Source 
fixes #4157
This commit is contained in:
Alex Lam S.L
2020-09-28 07:09:55 +01:00
committed by GitHub
parent 92d1391e5e
commit 9e07ac4102
2 changed files with 73 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
lib/compress.js
View File

@@ -4448,15 +4448,16 @@ merge(Compressor.prototype, {
push();
segment.block = node;
walk_body(node, tw);
pop();
if (node.bcatch) {
var def = node.bcatch.argname.definition();
references[def.id] = false;
if (def = def.redefined()) references[def.id] = false;
pop();
push();
if (node.bfinally) segment.block = node.bcatch;
walk_body(node.bcatch, tw);
pop();
}
pop();
if (node.bfinally) node.bfinally.walk(tw);
return true;
}

72
test/compress/merge_vars.js
View File

@@ -2617,9 +2617,9 @@ issue_4126_1: {
try {
console.log("PASS");
} catch (e) {
var c = a;
var b = a;
} finally {
var c = c;
var c = b;
}
console.log(c);
}
@@ -2860,3 +2860,71 @@ issue_4155: {
"function",
]
}
issue_4157_1: {
options = {
dead_code: true,
loops: true,
merge_vars: true,
}
input: {
(function() {
try {
for (var a = "FAIL"; a; a++)
return;
var b = 0;
} finally {
console.log(b);
}
})();
}
expect: {
(function() {
try {
var a = "FAIL";
if (a)
return;
var b = 0;
} finally {
console.log(b);
}
})();
}
expect_stdout: "undefined"
}
issue_4157_2: {
options = {
dead_code: true,
loops: true,
merge_vars: true,
}
input: {
(function() {
try {
throw "FAIL";
} catch (e) {
for (var a = e; a; a++)
return;
var b = 0;
} finally {
console.log(b);
}
})();
}
expect: {
(function() {
try {
throw "FAIL";
} catch (e) {
var a = e;
if (a)
return;
var b = 0;
} finally {
console.log(b);
}
})();
}
expect_stdout: "undefined"
}