[Pkg-javascript-commits] [uglifyjs] 303/491: convert to number under boolean context (#2545)

[Jonas Smedegaard]

This is an automated email from the git hooks/post-receive script.

js pushed a commit to annotated tag debian/3.3.10-1
in repository uglifyjs.

commit 09b320e8a5ec85410bf323573b76f2da7e08e2ee
Author: Alex Lam S.L <alexlamsl at gmail.com>
Date:   Fri Dec 1 12:52:36 2017 +0800

    convert to number under boolean context (#2545)
---
 lib/compress.js               | 3 +++
 test/compress/conditionals.js | 4 ++--
 test/compress/issue-1639.js   | 4 ++--
 test/compress/issue-637.js    | 2 +-
 test/compress/issue-640.js    | 2 +-
 test/compress/reduce_vars.js  | 2 +-
 test/compress/transform.js    | 6 +++---
 test/compress/typeof.js       | 4 ++--
 8 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/lib/compress.js b/lib/compress.js
index 395c4ed..8b2951d 100644
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -4699,6 +4699,9 @@ merge(Compressor.prototype, {
     });
 
     OPT(AST_Boolean, function(self, compressor){
+        if (compressor.in_boolean_context()) return make_node(AST_Number, self, {
+            value: +self.value
+        });
         if (compressor.option("booleans")) {
             var p = compressor.parent();
             if (p instanceof AST_Binary && (p.operator == "=="
diff --git a/test/compress/conditionals.js b/test/compress/conditionals.js
index 9cb27fa..22947d8 100644
--- a/test/compress/conditionals.js
+++ b/test/compress/conditionals.js
@@ -1037,10 +1037,10 @@ issue_2535: {
     expect: {
         y();
         x() && y();
-        (x(), 0) || y();
+        (x(), 1) && y();
         x() && y();
         x() && y();
         x() && y();
-        (x(), 1) || y();
+        (x(), 0) && y();
     }
 }
diff --git a/test/compress/issue-1639.js b/test/compress/issue-1639.js
index b6a9647..fc3db98 100644
--- a/test/compress/issue-1639.js
+++ b/test/compress/issue-1639.js
@@ -26,7 +26,7 @@ issue_1639_1: {
     }
     expect: {
         for (var a = 100, b = 10, L1 = 5; --L1 > 0;)
-            if (--b, !1) var ignore = 0;
+            if (--b, 0) var ignore = 0;
         console.log(a, b);
     }
     expect_stdout: true
@@ -57,7 +57,7 @@ issue_1639_2: {
     expect: {
         var a = 100, b = 10;
         function f19() {
-            ++a, 1;
+            ++a, 0;
         }
         f19(),
         console.log(a, b);
diff --git a/test/compress/issue-637.js b/test/compress/issue-637.js
index 45fd248..ed42cb8 100644
--- a/test/compress/issue-637.js
+++ b/test/compress/issue-637.js
@@ -17,6 +17,6 @@ wrongly_optimized: {
             foo();
         }
         // TODO: optimize to `func(), bar()`
-        (func(), 0) || bar();
+        (func(), 1) && bar();
     }
 }
diff --git a/test/compress/issue-640.js b/test/compress/issue-640.js
index c9a68dc..784092c 100644
--- a/test/compress/issue-640.js
+++ b/test/compress/issue-640.js
@@ -100,7 +100,7 @@ wrongly_optimized: {
             foo();
         }
         // TODO: optimize to `func(), bar()`
-        if (func(), !0) bar();
+        if (func(), 1) bar();
     }
 }
 
diff --git a/test/compress/reduce_vars.js b/test/compress/reduce_vars.js
index bcfa7b3..4a098f7 100644
--- a/test/compress/reduce_vars.js
+++ b/test/compress/reduce_vars.js
@@ -2347,7 +2347,7 @@ booleans: {
     }
     expect: {
         console.log(function(a) {
-            if (!1);
+            if (0);
             switch (!1) {
               case 0:
                 return "FAIL";
diff --git a/test/compress/transform.js b/test/compress/transform.js
index 1e21da5..68d3384 100644
--- a/test/compress/transform.js
+++ b/test/compress/transform.js
@@ -45,9 +45,9 @@ condition_evaluate: {
         if (void 0 == null);
     }
     expect: {
-        while (!1);
-        for (; !0;);
-        if (!0);
+        while (0);
+        for (; 1;);
+        if (1);
     }
 }
 
diff --git a/test/compress/typeof.js b/test/compress/typeof.js
index 60f3d1d..34949fb 100644
--- a/test/compress/typeof.js
+++ b/test/compress/typeof.js
@@ -44,7 +44,7 @@ typeof_in_boolean_context: {
         function f2() { return g(), "Yes"; }
         foo();
         console.log(1);
-        var a = !(console.log(2), !0);
+        var a = !(console.log(2), 1);
         foo();
     }
 }
@@ -57,6 +57,6 @@ issue_1668: {
         if (typeof bar);
     }
     expect: {
-        if (!0);
+        if (1);
     }
 }

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/uglifyjs.git