Skip to content

Commit

Permalink
fix corner case in inline (#5083)
Browse files Browse the repository at this point in the history
fixes #5082
  • Loading branch information
alexlamsl committed Jul 16, 2021
1 parent f18804f commit 902997b
Show file tree
Hide file tree
Showing 2 changed files with 61 additions and 5 deletions.
8 changes: 3 additions & 5 deletions lib/compress.js
Original file line number Diff line number Diff line change
Expand Up @@ -9522,11 +9522,9 @@ merge(Compressor.prototype, {
refs.forEach(function(ref) {
ref.scope = exp === fn ? fn.parent_scope : exp.scope;
ref.reference();
if (replacing) {
ref.definition().replaced++;
} else {
ref.definition().single_use = false;
}
var def = ref.definition();
if (replacing) def.replaced++;
def.single_use = false;
});
return node;
} else if (!node.has_side_effects(compressor)) {
Expand Down
58 changes: 58 additions & 0 deletions test/compress/classes.js
Original file line number Diff line number Diff line change
Expand Up @@ -2008,3 +2008,61 @@ issue_5053_4: {
expect_stdout: "PASS"
node_version: ">=4"
}

issue_5082_1: {
options = {
inline: true,
reduce_vars: true,
unused: true,
}
input: {
(function() {
class A {
p = console.log("PASS");
q() {}
}
class B {
static P = new A();
}
})();
}
expect: {
(function() {
class A {
p = console.log("PASS");
q() {}
}
new A();
})();
}
expect_stdout: "PASS"
node_version: ">=12"
}

issue_5082_2: {
options = {
inline: true,
passes: 2,
reduce_vars: true,
unused: true,
}
input: {
(function() {
class A {
p = console.log("PASS");
q() {}
}
class B {
static P = new A();
}
})();
}
expect: {
void new class {
p = console.log("PASS");
q() {}
}();
}
expect_stdout: "PASS"
node_version: ">=12"
}

0 comments on commit 902997b

Please sign in to comment.