From 83b124f0d652e618c6e3f0f57667c8a4a1224217 Mon Sep 17 00:00:00 2001 From: Jann Horn Date: Fri, 14 Oct 2016 18:04:39 +0200 Subject: [PATCH] fix uses_arguments handling (broken since 6605d1578351) Using the symbol declaration tracking of UglifyJS doesn't make sense here anyway, `arguments` always comes from something in the current scope. fixes #1299 --- lib/scope.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/scope.js b/lib/scope.js index fb583291..4943b568 100644 --- a/lib/scope.js +++ b/lib/scope.js @@ -204,6 +204,9 @@ AST_Toplevel.DEFMETHOD("figure_out_scope", function(options){ } } var sym = node.scope.find_variable(name); + if (node.scope instanceof AST_Lambda && name == "arguments") { + node.scope.uses_arguments = true; + } if (!sym) { var g; if (globals.has(name)) { @@ -215,9 +218,6 @@ AST_Toplevel.DEFMETHOD("figure_out_scope", function(options){ globals.set(name, g); } sym = g; - if (func && name == "arguments") { - func.uses_arguments = true; - } } node.thedef = sym; if (parent instanceof AST_Unary && (parent.operator === '++' || parent.operator === '--')