JoBoEVD/UglifyJS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix corner case in evaluate (#3654)

Browse Source 
fixes #3653
This commit is contained in:
Alex Lam S.L
2019-12-29 02:50:57 +00:00
committed by GitHub
parent 2f3930d1b9
commit af310ba2d0
2 changed files with 68 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

42
lib/compress.js
View File

@@ -6466,13 +6466,6 @@ merge(Compressor.prototype, {
}).optimize(compressor); }).optimize(compressor);
} }
break; break;
// 0 - n => -n
case "-":
if (self.left.value == 0) return make_node(AST_UnaryPrefix, self, {
operator: "-",
expression: self.right
}).optimize(compressor);
break;
// 1 * n => n // 1 * n => n
case "*": case "*":
if (self.left.value == 1) { if (self.left.value == 1) {
@@ -6483,15 +6476,32 @@ merge(Compressor.prototype, {
} }
break; break;
} }
// n - 0 => n if (self.right instanceof AST_Number && !self.left.is_constant()) switch (self.operator) {
// n / 1 => n // n + 0 => n
if (self.right instanceof AST_Number && !self.left.is_constant() && self.right.value == { case "+":
"-": 0, if (self.right.value == 0 && (self.left.is_boolean(compressor) || self.left.is_number(compressor))) {
"/": 1, return self.left;
}[self.operator]) return self.left.is_number(compressor) ? self.left : make_node(AST_UnaryPrefix, self, { }
operator: "+", break;
expression: self.left // n - 0 => n
}).optimize(compressor); case "-":
if (self.right.value == 0) {
return self.left.is_number(compressor) ? self.left : make_node(AST_UnaryPrefix, self, {
operator: "+",
expression: self.left
}).optimize(compressor);
}
break;
// n / 1 => n
case "/":
if (self.right.value == 1) {
return self.left.is_number(compressor) ? self.left : make_node(AST_UnaryPrefix, self, {
operator: "+",
expression: self.left
}).optimize(compressor);
}
break;
}
} }
if (compressor.option("typeofs")) switch (self.operator) { if (compressor.option("typeofs")) switch (self.operator) {
case "&&": case "&&":

45
test/compress/numbers.js
View File

@@ -997,7 +997,7 @@ identity_1: {
expect: { expect: {
0 + a; 0 + a;
a + 0; a + 0;
-a; 0 - a;
+a; +a;
+a; +a;
+a; +a;
@@ -1023,7 +1023,7 @@ identity_2: {
expect: { expect: {
+!a; +!a;
+!a; +!a;
-!a; 0 - !a;
+!a; +!a;
+!a; +!a;
+!a; +!a;
@@ -1049,7 +1049,7 @@ identity_3: {
expect: { expect: {
--a; --a;
--a; --a;
- --a; 0 - --a;
--a; --a;
--a; --a;
--a; --a;
@@ -1057,3 +1057,42 @@ identity_3: {
--a; --a;
} }
} }
issue_3653: {
options = {
evaluate: true,
}
input: {
console.log(0 - (console && 0));
console.log(0 + (0 - (console && 0)));
console.log(0 - (0 - (console && 0)));
console.log(1 * (0 - (console && 0)));
console.log(1 / (0 - (console && 0)));
console.log((0 - (console && 0)) + 0);
console.log((0 - (console && 0)) - 0);
console.log((0 - (console && 0)) * 1);
console.log((0 - (console && 0)) / 1);
}
expect: {
console.log(0 - (console && 0));
console.log(0 - (console && 0));
console.log(0 - (0 - (console && 0)));
console.log(0 - (console && 0));
console.log(1 / (0 - (console && 0)));
console.log(0 - (console && 0));
console.log(0 - (console && 0));
console.log(0 - (console && 0));
console.log(0 - (console && 0));
}
expect_stdout: [
"0",
"0",
"0",
"0",
"Infinity",
"0",
"0",
"0",
"0",
]
}