Merge pull request #3965 from glittershark/fix-top-level-ellipsis

Pass all args when auto-calling a function with an ellipsis
This commit is contained in:
Eelco Dolstra 2020-08-27 19:52:43 +02:00 committed by GitHub
commit f4d3b4fb76
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -1299,13 +1299,24 @@ void EvalState::autoCallFunction(Bindings & args, Value & fun, Value & res)
Value * actualArgs = allocValue();
mkAttrs(*actualArgs, fun.lambda.fun->formals->formals.size());
if (fun.lambda.fun->formals->ellipsis) {
// If the formals have an ellipsis (eg the function accepts extra args) pass
// all available automatic arguments (which includes arguments specified on
// the command line via --arg/--argstr)
for (auto& v : args) {
actualArgs->attrs->push_back(v);
}
} else {
// Otherwise, only pass the arguments that the function accepts
for (auto & i : fun.lambda.fun->formals->formals) {
Bindings::iterator j = args.find(i.name);
if (j != args.end())
if (j != args.end()) {
actualArgs->attrs->push_back(*j);
else if (!i.def)
} else if (!i.def) {
throwTypeError("cannot auto-call a function that has an argument without a default value ('%1%')", i.name);
}
}
}
actualArgs->attrs->sort();