fix corner case in evaluate (#5363)

fixes #5362
mishoo · Feb 20, 2022 · 212ce46 · 212ce46
1 parent fbc5ecf
commit 212ce46
Show file tree

Hide file tree

Showing 2 changed files with 41 additions and 2 deletions.
diff --git a/lib/compress.js b/lib/compress.js
@@ -10799,8 +10799,11 @@ Compressor.prototype.compress = function(node) {
         }
         switch (op) {
           case "+":
-            if (compressor.option("evaluate") && exp.is_number(compressor, true)) return exp;
-            break;
+            if (!compressor.option("evaluate")) break;
+            if (!exp.is_number(compressor, true)) break;
+            var parent = compressor.parent();
+            if (parent instanceof AST_UnaryPrefix && parent.operator == "delete") break;
+            return exp;
           case "-":
             if (exp instanceof AST_Infinity) exp = exp.transform(compressor);
             // avoids infinite recursion of numerals

diff --git a/test/compress/evaluate.js b/test/compress/evaluate.js
@@ -3294,3 +3294,39 @@ issue_5356: {
     }
     expect_stdout: "NaN"
 }
+
+issue_5362_1: {
+    options = {
+        evaluate: true,
+        reduce_vars: true,
+        toplevel: true,
+    }
+    input: {
+        var a = -console;
+        console.log(delete +a);
+    }
+    expect: {
+        var a = -console;
+        console.log((+a, true));
+    }
+    expect_stdout: "true"
+}
+
+issue_5362_2: {
+    options = {
+        evaluate: true,
+        reduce_vars: true,
+        side_effects: true,
+        toplevel: true,
+        unsafe: true,
+        unused: true,
+    }
+    input: {
+        var a = -console;
+        console.log(delete +a);
+    }
+    expect: {
+        console.log(true);
+    }
+    expect_stdout: "true"
+}