restore `inline` functionality disabled by #4204 (#4209)
authorAlex Lam S.L <alexlamsl@gmail.com>
Tue, 13 Oct 2020 01:33:49 +0000 (02:33 +0100)
committerGitHub <noreply@github.com>
Tue, 13 Oct 2020 01:33:49 +0000 (09:33 +0800)
lib/compress.js

index c59e26e..2307abc 100644 (file)
@@ -7069,7 +7069,7 @@ merge(Compressor.prototype, {
                         return node;
                     }
                 }
-                var scope, in_loop, level = -1;
+                var child, in_loop, scope;
                 if (replacing && can_inject_symbols()) {
                     fn._squeezed = true;
                     if (exp !== fn) fn.parent_scope = exp.scope;
@@ -7239,14 +7239,14 @@ merge(Compressor.prototype, {
 
         function can_inject_symbols() {
             var defined = Object.create(null);
-            var child;
+            var level = 0;
             scope = compressor.self();
-            do {
-                child = scope;
-                scope = compressor.parent(++level);
+            while (!(scope instanceof AST_Scope)) {
                 if (scope.variables) scope.variables.each(function(def) {
                     defined[def.name] = true;
                 });
+                child = scope;
+                scope = compressor.parent(level++);
                 if (scope instanceof AST_DWLoop) {
                     in_loop = [];
                 } else if (scope instanceof AST_For) {
@@ -7259,7 +7259,7 @@ merge(Compressor.prototype, {
                 } else if (scope instanceof AST_SymbolRef) {
                     if (scope.fixed_value() instanceof AST_Scope) return false;
                 }
-            } while (!(scope instanceof AST_Scope));
+            }
             var safe_to_inject = (!(scope instanceof AST_Toplevel) || compressor.toplevel.vars)
                 && (exp !== fn || fn.parent_scope.resolve() === compressor.find_parent(AST_Scope));
             var inline = compressor.option("inline");
@@ -7362,7 +7362,7 @@ merge(Compressor.prototype, {
                     return true;
                 }
             });
-            args.unshift(scope.body.indexOf(compressor.parent(level - 1)) + 1, 0);
+            args.unshift(scope.body.indexOf(child) + 1, 0);
             if (decls.length) args.push(make_node(AST_Var, fn, {
                 definitions: decls
             }));