From 3096f6fdad2d92b822ab75bbe78903c152e6b844 Mon Sep 17 00:00:00 2001 From: "Alex Lam S.L" Date: Tue, 13 Oct 2020 02:33:49 +0100 Subject: [PATCH] restore `inline` functionality disabled by #4204 (#4209) --- lib/compress.js | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/lib/compress.js b/lib/compress.js index c59e26ed..2307abc7 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -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 })); -- 2.34.1