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

fix corner case in ie8

Browse Source 
fixes #5039
This commit is contained in:
alexlamsl 2021-06-29 20:25:30 +08:00
parent 798121c9f3
commit 1271c85cc4
2 changed files with 29 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
lib/compress.js
View File

@ -7861,11 +7861,13 @@ merge(Compressor.prototype, {
var prop = this.property;
if (expr.may_throw_on_access(compressor)) {
if (!this.optional) return this;
prop = prop.drop_side_effect_free(compressor);
if (!prop) return expr.drop_side_effect_free(compressor, first_in_statement);
var node = this.clone();
node.property = prop;
return node;
if (prop.has_side_effects(compressor)) {
prop = prop.drop_side_effect_free(compressor);
if (!prop) return expr.drop_side_effect_free(compressor, first_in_statement);
var node = this.clone();
node.property = prop;
return node;
}
}
expr = expr.drop_side_effect_free(compressor, first_in_statement);
if (!expr) return prop.drop_side_effect_free(compressor, first_in_statement);

22
test/compress/optional-chains.js
View File

@ -305,3 +305,25 @@ issue_4947_2: {
expect_stdout: "PASS"
node_version: ">=14"
}
issue_5039: {
options = {
ie8: true,
side_effects: true,
toplevel: true,
unused: true,
}
input: {
var a = a?.[function f() {
f;
a;
}];
console.log("PASS");
}
expect: {
(function f() {});
console.log("PASS");
}
expect_stdout: "PASS"
node_version: ">=14"
}