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

fix corner case in inline (#5532)

Browse Source 
fixes #5531
This commit is contained in:
Alex Lam S.L
2022-06-29 21:09:53 +01:00
committed by GitHub
parent e1b03d0235
commit 2426657daa
2 changed files with 113 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lib/compress.js
View File

@@ -10784,7 +10784,9 @@ Compressor.prototype.compress = function(node) {
});
child = scope;
scope = compressor.parent(level++);
if (scope instanceof AST_DWLoop) {
if (scope instanceof AST_ClassField) {
if (!scope.static) return false;
} else if (scope instanceof AST_DWLoop) {
in_loop = [];
} else if (scope instanceof AST_For) {
if (scope.init === child) continue;

110
test/compress/classes.js
View File

@@ -3331,3 +3331,113 @@ issue_5512: {
expect_stdout: "PASS"
node_version: ">=16"
}
issue_5531_1: {
options = {
inline: true,
toplevel: true,
}
input: {
class A {
p = function() {
var a = function f() {
if (!a)
console.log("foo");
return 42;
}(a++);
}();
}
new A();
new A();
}
expect: {
class A {
p = function() {
var a = function f() {
if (!a)
console.log("foo");
return 42;
}(a++);
}();
}
new A();
new A();
}
expect_stdout: [
"foo",
"foo",
]
node_version: ">=12"
}
issue_5531_2: {
options = {
inline: true,
toplevel: true,
}
input: {
class A {
static p = function() {
var a = function f() {
if (!a)
console.log("foo");
return 42;
}(a++);
}();
}
new A();
new A();
}
expect: {
class A {
static p = (a = function f() {
if (!a)
console.log("foo");
return 42;
}(a++), void 0);
}
var a;
new A();
new A();
}
expect_stdout: "foo"
node_version: ">=12"
}
issue_5531_3: {
options = {
inline: true,
}
input: {
class A {
static {
(function() {
var a = function f() {
if (!a)
console.log("foo");
return 42;
}(a++);
})();
}
}
new A();
new A();
}
expect: {
class A {
static {
a = function f() {
if (!a)
console.log("foo");
return 42;
}(a++),
void 0;
var a;
}
}
new A();
new A();
}
expect_stdout: "foo"
node_version: ">=16"
}