From: Alex Lam S.L <alexlamsl@gmail.com>
Date: Tue, 2 Mar 2021 07:27:55 +0000 (+0000)
Subject: enhance `reduce_vars` & `varify` (#4714)
X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=74dee5c445b1424866ef78ac9b06c507ef92959c;p=UglifyJS.git

enhance `reduce_vars` & `varify` (#4714)
---

diff --git a/lib/compress.js b/lib/compress.js
index 2d826697..393e3197 100644
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -1259,10 +1259,13 @@ merge(Compressor.prototype, {
         });
         def(AST_VarDef, function(tw, descend, compressor) {
             var node = this;
-            if (!node.value) return;
-            node.value.walk(tw);
+            if (node.value) {
+                node.value.walk(tw);
+            } else if (!(tw.parent() instanceof AST_Let)) {
+                return;
+            }
             scan_declaration(tw, compressor, node.name, function() {
-                return node.value;
+                return node.value || make_node(AST_Undefined, node);
             }, function(name, fixed) {
                 var d = name.definition();
                 if (fixed && safe_to_assign(tw, d, true)) {
diff --git a/test/compress/let.js b/test/compress/let.js
index 04002e9a..53956c48 100644
--- a/test/compress/let.js
+++ b/test/compress/let.js
@@ -610,6 +610,28 @@ drop_unused: {
     node_version: ">=4"
 }
 
+default_init: {
+    options = {
+        evaluate: true,
+        reduce_vars: true,
+        sequences: true,
+        toplevel: true,
+        unused: true,
+    }
+    input: {
+        "use strict";
+        let a;
+        a = "PASS";
+        console.log(a);
+    }
+    expect: {
+        "use strict";
+        console.log("PASS");
+    }
+    expect_stdout: "PASS"
+    node_version: ">=4"
+}
+
 issue_4191: {
     options = {
         functions: true,
diff --git a/test/compress/varify.js b/test/compress/varify.js
index 4d2a6b5a..ab212842 100644
--- a/test/compress/varify.js
+++ b/test/compress/varify.js
@@ -409,3 +409,31 @@ drop_forin_let: {
     expect_stdout: "PASS"
     node_version: ">=4"
 }
+
+default_init: {
+    options = {
+        join_vars: true,
+        reduce_vars: true,
+        unused: true,
+        varify: true,
+    }
+    input: {
+        A = "PASS";
+        (function() {
+            "use strict";
+            let a;
+            a = A;
+            console.log(a);
+        })();
+    }
+    expect: {
+        A = "PASS";
+        (function() {
+            "use strict";
+            var a = A;
+            console.log(a);
+        })();
+    }
+    expect_stdout: "PASS"
+    node_version: ">=4"
+}