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

Only compress code in new Function if all arguments are strings.

Browse Source
This commit is contained in:
Mihai Bazon
2013-05-14 18:36:31 +03:00
parent d13aa3954d
commit caa8896a8a
2 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/compress.js
View File

@@ -1590,7 +1590,7 @@ merge(Compressor.prototype, {
right: make_node(AST_String, self, { value: "" }) right: make_node(AST_String, self, { value: "" })
}); });
case "Function": case "Function":
if (self.args[self.args.length - 1] instanceof AST_String) { if (all(self.args, function(x){ return x instanceof AST_String })) {
// quite a corner-case, but we can handle it: // quite a corner-case, but we can handle it:
// https://github.com/mishoo/UglifyJS2/issues/203 // https://github.com/mishoo/UglifyJS2/issues/203
// if the code argument is a constant, then we can minify it. // if the code argument is a constant, then we can minify it.

7
lib/utils.js
View File

@@ -245,6 +245,13 @@ function makePredicate(words) {
return new Function("str", f); return new Function("str", f);
}; };
function all(array, predicate) {
for (var i = array.length; --i >= 0;)
if (!predicate(array[i]))
return false;
return true;
};
function Dictionary() { function Dictionary() {
this._values = Object.create(null); this._values = Object.create(null);
this._size = 0; this._size = 0;