scan RHS of dropped assignments (#1581)

- similar case as #1578 but against #1450 instead
- fix `this` binding in #1450 as well

closes #1580
This commit is contained in:
Alex Lam S.L
2017-03-09 05:22:27 +08:00
committed by GitHub
parent aa80ee349d
commit 7e465d4a01
2 changed files with 44 additions and 10 deletions

View File

@@ -1901,17 +1901,15 @@ merge(Compressor.prototype, {
}
return node;
}
if (drop_vars && assign_as_unused) {
var n = node;
while (n instanceof AST_Assign
&& n.operator == "="
&& n.left instanceof AST_SymbolRef) {
var def = n.left.definition();
if (def.id in in_use_ids
|| self.variables.get(def.name) !== def) break;
n = n.right;
if (drop_vars && assign_as_unused
&& node instanceof AST_Assign
&& node.operator == "="
&& node.left instanceof AST_SymbolRef) {
var def = node.left.definition();
if (!(def.id in in_use_ids)
&& self.variables.get(def.name) === def) {
return maintain_this_binding(tt.parent(), node, node.right.transform(tt));
}
if (n !== node) return n;
}
if (node instanceof AST_For) {
descend(node, this);

View File

@@ -725,3 +725,39 @@ vardef_value: {
}
}
}
assign_binding: {
options = {
cascade: true,
side_effects: true,
unused: true,
}
input: {
function f() {
var a;
a = f.g, a();
}
}
expect: {
function f() {
(0, f.g)();
}
}
}
assign_chain: {
options = {
unused: true,
}
input: {
function f() {
var a, b;
x = a = y = b = 42;
}
}
expect: {
function f() {
x = y = 42;
}
}
}