fix corner case in `functions` (#3367)

fixes #3366

diff --git a/lib/compress.js b/lib/compress.js
index f5479be..f72972b 100644 (file)
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -3602,7 +3602,7 @@ merge(Compressor.prototype, {
                             compressor.warn("Declaring {name} as function [{file}:{line},{col}]", template(def.name));
                             var defun = make_node(AST_Defun, def, def.value);
                             defun.name = make_node(AST_SymbolDefun, def.name, def.name);
-                            defun.parent_scope.resolve().def_function(defun.name);
+                            def.name.scope.resolve().def_function(defun.name);
                             body.push(defun);
                         } else {
                             if (side_effects.length > 0) {

diff --git a/test/compress/functions.js b/test/compress/functions.js
index c43c17e..7303b91 100644 (file)
--- a/test/compress/functions.js
+++ b/test/compress/functions.js
@@ -2981,3 +2981,34 @@ issue_3364: {
     }
     expect_stdout: "2"
 }
+
+issue_3366: {
+    options = {
+        functions: true,
+        inline: true,
+        reduce_vars: true,
+        toplevel: true,
+        unused: true,
+    }
+    input: {
+        function f() {
+            function g() {
+                return function() {};
+            }
+            var a = g();
+            (function() {
+                this && a && console.log("PASS");
+            })();
+        }
+        f();
+    }
+    expect: {
+        (function() {
+            function a() {}
+            (function() {
+                this && a && console.log("PASS");
+            })();
+        })();
+    }
+    expect_stdout: "PASS"
+}

diff --git a/test/sandbox.js b/test/sandbox.js
index 23fb4fc..396a08f 100644 (file)
--- a/test/sandbox.js
+++ b/test/sandbox.js
@@ -73,8 +73,10 @@ exports.run_code = function(code, reuse) {
         process.stdout.write = original_write;
         if (!reuse || code.indexOf(".prototype") >= 0) {
             context = null;
-        } else for (var key in context) {
-            delete context[key];
+        } else {
+            vm.runInContext(Object.keys(context).map(function(name) {
+                return "delete " + name;
+            }).join("\n"), context);
         }
     }
 };