fix inline handling of AST_Call.args (#2059)
This commit is contained in:
@@ -3183,19 +3183,21 @@ merge(Compressor.prototype, {
|
|||||||
if (body) {
|
if (body) {
|
||||||
var fn = exp.clone();
|
var fn = exp.clone();
|
||||||
fn.argnames = [];
|
fn.argnames = [];
|
||||||
fn.body = [
|
fn.body = [];
|
||||||
make_node(AST_Var, self, {
|
if (exp.argnames.length > 0) {
|
||||||
|
fn.body.push(make_node(AST_Var, self, {
|
||||||
definitions: exp.argnames.map(function(sym, i) {
|
definitions: exp.argnames.map(function(sym, i) {
|
||||||
|
var arg = self.args[i];
|
||||||
return make_node(AST_VarDef, sym, {
|
return make_node(AST_VarDef, sym, {
|
||||||
name: sym,
|
name: sym,
|
||||||
value: self.args[i] || make_node(AST_Undefined, self)
|
value: arg ? arg.clone(true) : make_node(AST_Undefined, self)
|
||||||
});
|
});
|
||||||
})
|
})
|
||||||
})
|
}));
|
||||||
];
|
}
|
||||||
if (self.args.length > exp.argnames.length) {
|
if (self.args.length > exp.argnames.length) {
|
||||||
fn.body.push(make_node(AST_SimpleStatement, self, {
|
fn.body.push(make_node(AST_SimpleStatement, self, {
|
||||||
body: make_sequence(self, self.args.slice(exp.argnames.length))
|
body: make_sequence(self, self.args.slice(exp.argnames.length)).clone(true)
|
||||||
}));
|
}));
|
||||||
}
|
}
|
||||||
fn.body.push(make_node(AST_Return, self, {
|
fn.body.push(make_node(AST_Return, self, {
|
||||||
|
|||||||
@@ -431,3 +431,53 @@ pure_annotation: {
|
|||||||
}
|
}
|
||||||
expect_exact: ""
|
expect_exact: ""
|
||||||
}
|
}
|
||||||
|
|
||||||
|
drop_fargs: {
|
||||||
|
options = {
|
||||||
|
cascade: true,
|
||||||
|
inline: true,
|
||||||
|
keep_fargs: false,
|
||||||
|
side_effects: true,
|
||||||
|
unused: true,
|
||||||
|
}
|
||||||
|
input: {
|
||||||
|
var a = 1;
|
||||||
|
!function(a_1) {
|
||||||
|
a++;
|
||||||
|
}(a++ + (a && a.var));
|
||||||
|
console.log(a);
|
||||||
|
}
|
||||||
|
expect: {
|
||||||
|
var a = 1;
|
||||||
|
!function() {
|
||||||
|
a++;
|
||||||
|
}(++a && a.var);
|
||||||
|
console.log(a);
|
||||||
|
}
|
||||||
|
expect_stdout: "3"
|
||||||
|
}
|
||||||
|
|
||||||
|
keep_fargs: {
|
||||||
|
options = {
|
||||||
|
cascade: true,
|
||||||
|
inline: true,
|
||||||
|
keep_fargs: true,
|
||||||
|
side_effects: true,
|
||||||
|
unused: true,
|
||||||
|
}
|
||||||
|
input: {
|
||||||
|
var a = 1;
|
||||||
|
!function(a_1) {
|
||||||
|
a++;
|
||||||
|
}(a++ + (a && a.var));
|
||||||
|
console.log(a);
|
||||||
|
}
|
||||||
|
expect: {
|
||||||
|
var a = 1;
|
||||||
|
!function(a_1) {
|
||||||
|
a++;
|
||||||
|
}(++a && a.var);
|
||||||
|
console.log(a);
|
||||||
|
}
|
||||||
|
expect_stdout: "3"
|
||||||
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user