Skip to content

Commit

Permalink
fix corner case in unused (#3665)
Browse files Browse the repository at this point in the history
fixes #3664
  • Loading branch information
alexlamsl committed Jan 1, 2020
1 parent d83d3d7 commit 746f5f6
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lib/compress.js
Expand Up @@ -3913,7 +3913,7 @@ merge(Compressor.prototype, {
if (node instanceof AST_Assign) {
if (!in_use || node.left === sym && def.id in fixed_ids && fixed_ids[def.id] !== node) {
value = get_rhs(node);
if (node.write_only) {
if (node.write_only === true) {
value = value.drop_side_effect_free(compressor) || make_node(AST_Number, node, {
value: 0
});
Expand Down
20 changes: 20 additions & 0 deletions test/compress/drop-unused.js
Expand Up @@ -2335,3 +2335,23 @@ function_parameter_ie8: {
}
expect_stdout: "PASS"
}

issue_3664: {
options = {
pure_getters: "strict",
unused: true,
}
input: {
console.log(function() {
var a, b = (a = (a = [ b && console.log("FAIL") ]).p = 0, 0);
return "PASS";
}());
}
expect: {
console.log(function() {
var b = ([ b && console.log("FAIL") ].p = 0, 0);
return "PASS";
}());
}
expect_stdout: "PASS"
}

0 comments on commit 746f5f6

Please sign in to comment.