From: Alex Lam S.L <alexlamsl@gmail.com>
Date: Fri, 25 Sep 2020 00:04:51 +0000 (+0100)
Subject: fix corner case in `merge_vars` (#4151)
X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=af35cd32f28dfc3f79380d8f1b8294fc550d07e2;p=UglifyJS.git

fix corner case in `merge_vars` (#4151)
---

diff --git a/lib/compress.js b/lib/compress.js
index 883345b6..517f29fd 100644
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -4331,7 +4331,7 @@ merge(Compressor.prototype, {
 
     AST_Scope.DEFMETHOD("merge_variables", function(compressor) {
         if (!compressor.option("merge_vars")) return;
-        var self = this, segment = {};
+        var self = this, segment = {}, root;
         var first = [], last = [], index = 0;
         var declarations = new Dictionary();
         var references = Object.create(null);
@@ -4414,10 +4414,8 @@ merge(Compressor.prototype, {
             if (node instanceof AST_Scope) {
                 push();
                 segment.block = node;
-                if (node instanceof AST_Lambda && node.name) {
-                    if (node !== self) segment.loop = true;
-                    references[node.name.definition().id] = false;
-                }
+                if (node === self) root = segment;
+                if (node instanceof AST_Lambda && node.name) references[node.name.definition().id] = false;
                 descend();
                 pop();
                 return true;
@@ -4563,7 +4561,8 @@ merge(Compressor.prototype, {
                         definition: def,
                     });
                 }
-                refs.start = self;
+                if (segment.block !== self) return references[def.id] = false;
+                refs.start = root;
             }
             prev[def.id] = last.length;
             last.push({
diff --git a/test/compress/merge_vars.js b/test/compress/merge_vars.js
index 559832c2..b5148669 100644
--- a/test/compress/merge_vars.js
+++ b/test/compress/merge_vars.js
@@ -2766,3 +2766,37 @@ issue_4139: {
     }
     expect_stdout: "object"
 }
+
+lambda_reuse: {
+    options = {
+        merge_vars: true,
+        toplevel: true,
+    }
+    input: {
+        var a, b, f = function() {
+            console.log(a);
+        };
+        f();
+        a = "PASS";
+        b = "FAIL";
+        f();
+        if (console.log(typeof b))
+            console.log(b);
+    }
+    expect: {
+        var a, b, f = function() {
+            console.log(a);
+        };
+        f();
+        a = "PASS";
+        b = "FAIL";
+        f();
+        if (console.log(typeof b))
+            console.log(b);
+    }
+    expect_stdout: [
+        "undefined",
+        "PASS",
+        "string",
+    ]
+}