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

fix corner case in merge_vars (#4302)

Browse Source 
fixes #4301
This commit is contained in:
Alex Lam S.L
2020-11-18 21:44:47 +00:00
committed by GitHub
parent fd6544b340
commit aecbabc587
2 changed files with 48 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
lib/compress.js
View File

@@ -4635,14 +4635,27 @@ merge(Compressor.prototype, {
var lhs = node.left;
if (lhs instanceof AST_Destructured) {
node.right.walk(tw);
lhs.mark_symbol(function(node) {
if (node instanceof AST_SymbolRef) {
var marker = new TreeWalker(function(node) {
if (node instanceof AST_Destructured) return;
if (node instanceof AST_DestructuredKeyVal) {
if (node.key instanceof AST_Node) {
push();
node.key.walk(tw);
pop();
push();
node.value.walk(marker);
pop();
} else {
node.value.walk(marker);
}
} else if (node instanceof AST_SymbolRef) {
mark(node, false, true);
} else {
node.walk(tw);
}
return true;
}, tw);
});
lhs.walk(marker);
return true;
}
if (lhs instanceof AST_SymbolRef) {

32
test/compress/destructured.js
View File

@@ -1559,3 +1559,35 @@ issue_4298: {
expect_stdout: "PASS"
node_version: ">=6"
}
issue_4301: {
options = {
merge_vars: true,
}
input: {
try {
console.log(function() {
var a, b = console;
return {
[a = b]: a.p,
} = "foo";
}());
} catch (e) {
console.log("bar");
}
}
expect: {
try {
console.log(function() {
var a, b = console;
return {
[a = b]: a.p,
} = "foo";
}());
} catch (e) {
console.log("bar");
}
}
expect_stdout: true
node_version: ">=6"
}