aboutsummaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorAlex Lam S.L <alexlamsl@gmail.com>2021-03-03 01:18:02 +0000
committerGitHub <noreply@github.com>2021-03-03 09:18:02 +0800
commit10ca578ee57a0630c2b59d3da9b2c6e5ec6d476a (patch)
treedf7f6344cc225abdf186cece9ce0fda9b75c2efc /lib
parent955411e0657cbaea3226537b2738307d10754b02 (diff)
downloadtracifyjs-10ca578ee57a0630c2b59d3da9b2c6e5ec6d476a.tar.gz
tracifyjs-10ca578ee57a0630c2b59d3da9b2c6e5ec6d476a.zip
fix corner case in `inline` (#4726)
fixes #4725
Diffstat (limited to 'lib')
-rw-r--r--lib/ast.js2
-rw-r--r--lib/compress.js6
2 files changed, 7 insertions, 1 deletions
diff --git a/lib/ast.js b/lib/ast.js
index b035b5c7..e46fb2e3 100644
--- a/lib/ast.js
+++ b/lib/ast.js
@@ -1684,6 +1684,8 @@ var AST_ObjectMethod = DEFNODE("ObjectMethod", null, {
$documentation: "A key(){} object property",
_validate: function() {
if (!(this.value instanceof AST_LambdaExpression)) throw new Error("value must be AST_LambdaExpression");
+ if (is_arrow(this.value)) throw new Error("value cannot be AST_Arrow or AST_AsyncArrow");
+ if (this.value.name != null) throw new Error("name of class method's lambda must be null");
},
}, AST_ObjectKeyVal);
diff --git a/lib/compress.js b/lib/compress.js
index 55ff50be..1b22d861 100644
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -5210,6 +5210,7 @@ merge(Compressor.prototype, {
OPT(AST_Function, function(self, compressor) {
drop_rest_farg(self, compressor);
self.body = tighten_body(self.body, compressor);
+ var parent = compressor.parent();
if (compressor.option("inline")) for (var i = 0; i < self.body.length; i++) {
var stat = self.body[i];
if (stat instanceof AST_Directive) continue;
@@ -5231,6 +5232,9 @@ merge(Compressor.prototype, {
return def.scope !== self;
})) break;
}
+ if (fn.name
+ && (parent instanceof AST_ClassMethod || parent instanceof AST_ObjectMethod)
+ && parent.value === compressor.self()) break;
if (fn.contains_this()) break;
var len = fn.argnames.length;
if (len > 0 && compressor.option("inline") < 2) break;
@@ -5251,7 +5255,7 @@ merge(Compressor.prototype, {
if (call.args[j].has_side_effects(compressor)) break;
}
if (j < call.args.length) break;
- if (len < self.argnames.length && !compressor.drop_fargs(self, compressor.parent())) {
+ if (len < self.argnames.length && !compressor.drop_fargs(self, parent)) {
if (!compressor.drop_fargs(fn, call)) break;
do {
fn.argnames.push(fn.make_var(AST_SymbolFunarg, fn, "argument_" + len));