aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTal Ater <tal@talater.com>2014-09-13 18:59:19 +0300
committerRichard van Velzen <rvanvelzen1@gmail.com>2015-01-13 18:27:21 +0100
commita1a4c2ada70760851c63a7e3ff51ee7f786d3e8f (patch)
tree11e939a2b41bcfaa31b4abc162c23ccbbc930386
parent189dbf02b6f0c1abc0c10f092d72f7ffcc58a48f (diff)
downloadtracifyjs-a1a4c2ada70760851c63a7e3ff51ee7f786d3e8f.tar.gz
tracifyjs-a1a4c2ada70760851c63a7e3ff51ee7f786d3e8f.zip
Optimize conditionals where the consequent and alternative are both booleans and not equivalent
-rw-r--r--lib/compress.js14
-rw-r--r--test/compress/conditionals.js54
2 files changed, 68 insertions, 0 deletions
diff --git a/lib/compress.js b/lib/compress.js
index 72e4d92d..ae20d485 100644
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -2386,6 +2386,20 @@ merge(Compressor.prototype, {
}
}
+ // x=y?true:false --> x=!!y
+ if (consequent instanceof AST_True
+ && alternative instanceof AST_False) {
+ self.condition = self.condition.negate(compressor);
+ return make_node(AST_UnaryPrefix, self.condition, {
+ operator: "!",
+ expression: self.condition
+ });
+ }
+ // x=y?false:true --> x=!y
+ if (consequent instanceof AST_False
+ && alternative instanceof AST_True) {
+ return self.condition.negate(compressor)
+ }
return self;
});
diff --git a/test/compress/conditionals.js b/test/compress/conditionals.js
index 2212f85f..9af1630a 100644
--- a/test/compress/conditionals.js
+++ b/test/compress/conditionals.js
@@ -312,3 +312,57 @@ cond_7_1: {
x = (y, 2);
}
}
+
+cond_8: {
+ options = {
+ conditionals: true,
+ evaluate : true
+ };
+ input: {
+ // compress these
+ a = condition ? true : false;
+
+ a = !condition ? true : false;
+
+ a = condition() ? true : false;
+
+ if (condition) {
+ a = true;
+ } else {
+ a = false;
+ }
+
+ a = condition ? false : true;
+
+ a = !condition ? false : true;
+
+ a = condition() ? false : true;
+
+ if (condition) {
+ a = false;
+ } else {
+ a = true;
+ }
+
+ // don't compress these
+ a = condition ? 1 : false;
+
+ a = !condition ? true : 0;
+
+ a = condition ? 1 : 0;
+
+ }
+ expect: {
+ a = !!condition;
+ a = !condition;
+ a = !!condition();
+ a = !!condition;
+ a = !condition;
+ a = !!condition;
+ a = !condition();
+ a = !condition;
+ a = condition ? 1 : false;
+ a = condition ? 0 : true;
+ a = condition ? 1 : 0;
+ }
+} \ No newline at end of file