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

improve unsafe_Func (#2171)

Browse Source 
- minimise disturbance to `compute_char_frequency()`
- remove extraneous quotation marks
This commit is contained in:
Alex Lam S.L
2017-06-27 23:53:42 +08:00
committed by GitHub
parent 1e4de2e6d3
commit f0a99125ee
2 changed files with 12 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
lib/compress.js
View File

@@ -3120,7 +3120,7 @@ merge(Compressor.prototype, {
// 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.
try { try {
var code = "NaN(function(" + self.args.slice(0, -1).map(function(arg) { var code = "n(function(" + self.args.slice(0, -1).map(function(arg) {
return arg.value; return arg.value;
}).join(",") + "){" + self.args[self.args.length - 1].value + "})"; }).join(",") + "){" + self.args[self.args.length - 1].value + "})";
var ast = parse(code); var ast = parse(code);
@@ -3140,18 +3140,18 @@ merge(Compressor.prototype, {
return true; return true;
} }
})); }));
var args = fun.argnames.map(function(arg, i) {
return make_node(AST_String, self.args[i], {
value: arg.print_to_string()
});
});
var code = OutputStream(); var code = OutputStream();
AST_BlockStatement.prototype._codegen.call(fun, fun, code); AST_BlockStatement.prototype._codegen.call(fun, fun, code);
code = code.toString().replace(/^\{|\}$/g, ""); self.args = [
args.push(make_node(AST_String, self.args[self.args.length - 1], { make_node(AST_String, self, {
value: code value: fun.argnames.map(function(arg) {
})); return arg.print_to_string();
self.args = args; }).join(",")
}),
make_node(AST_String, self.args[self.args.length - 1], {
value: code.get().replace(/^\{|\}$/g, "")
})
];
return self; return self;
} catch (ex) { } catch (ex) {
if (ex instanceof JS_Parse_Error) { if (ex instanceof JS_Parse_Error) {

2
test/compress/functions.js
View File

@@ -265,7 +265,7 @@ issue_203: {
} }
expect: { expect: {
var m = {}; var m = {};
var fn = Function("n", "o", "o.exports=42"); var fn = Function("n,o", "o.exports=42");
fn(null, m, m.exports); fn(null, m, m.exports);
console.log(m.exports); console.log(m.exports);
} }