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

Reverted previous addition of compression of conditionals where the condition and consequent are equivalent.

Browse Source
This commit is contained in:
Tal Ater 2014-09-27 02:49:10 +03:00
parent cd44892301
commit a16d1d1d12
2 changed files with 0 additions and 45 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
lib/compress.js
View File

@ -2332,15 +2332,6 @@ merge(Compressor.prototype, {
}
}
// x=y?y:z --> x=y||z
if (self.condition.equivalent_to(consequent)
&& !self.condition.has_side_effects(compressor)) {
return make_node(AST_Binary, self.condition, {
left: self.condition,
operator: "||",
right: alternative
});
}
// x=y?true:false --> x=!!y
if (consequent instanceof AST_True
&& alternative instanceof AST_False) {

36
test/compress/conditionals.js
View File

@ -294,42 +294,6 @@ cond_7: {
}
cond_8: {
options = {
conditionals: true,
evaluate : true
};
input: {
// compress these
a = condition ? condition : b;
if (condition) {
a = condition;
} else {
a = b;
}
a = condition ? condition : b();
// Don't compress conditions that have side effects
if (condition()) {
a = condition();
} else {
a = b;
}
a = condition() ? condition() : b;
}
expect: {
a = condition || b;
a = condition || b;
a = condition || b();
a = condition() ? condition() : b;
a = condition() ? condition() : b;
}
}
cond_9: {
options = {
conditionals: true,
evaluate : true