fix corner case in merge_vars (#4104)

fixes #4103
This commit is contained in:
Alex Lam S.L
2020-09-15 12:47:12 +01:00
committed by GitHub
parent d64d0b0bec
commit 335456cf77
2 changed files with 29 additions and 1 deletions

View File

@@ -4315,10 +4315,11 @@ merge(Compressor.prototype, {
var prev = Object.create(null);
var tw = new TreeWalker(function(node, descend) {
if (node instanceof AST_Assign) {
if (node.operator != "=") return;
var sym = node.left;
if (!(sym instanceof AST_SymbolRef)) return;
node.right.walk(tw);
mark(sym, node.operator == "=");
mark(sym, true);
return true;
}
if (node instanceof AST_Binary) {

View File

@@ -260,3 +260,30 @@ read_before_assign_2: {
}
expect_stdout: "PASS"
}
issue_4103: {
options = {
merge_vars: true,
side_effects: true,
toplevel: true,
}
input: {
function f(a) {
console.log(a);
}
var b = 0;
var c = f(b++ + (c %= 1 >> console.log(c = 0)));
b;
}
expect: {
function f(a) {
console.log(a);
}
var b = 0;
var c = f(b++ + (c %= 1 >> console.log(c = 0)));
}
expect_stdout: [
"0",
"NaN",
]
}