diff options
author | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2013-05-16 17:56:14 +0200 |
---|---|---|
committer | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2013-05-16 17:56:14 +0200 |
commit | 1b3a03f1610b714adca41637ccd85a8157e236ab (patch) | |
tree | cfffb0927fe7ecb37018b60ae88a80857f8dcb43 /src/libexpr | |
parent | 229567293c4e0f31bc8c79f69b2ff25f8f6e5147 (diff) |
Show which function argument was unexpected
Fixes #116.
Diffstat (limited to 'src/libexpr')
-rw-r--r-- | src/libexpr/eval.cc | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/src/libexpr/eval.cc b/src/libexpr/eval.cc index 2f0c7e5cd..551ed4ba5 100644 --- a/src/libexpr/eval.cc +++ b/src/libexpr/eval.cc @@ -765,11 +765,15 @@ void EvalState::callFunction(Value & fun, Value & arg, Value & v) } /* Check that each actual argument is listed as a formal - argument (unless the attribute match specifies a `...'). - TODO: show the names of the expected/unexpected - arguments. */ - if (!fun.lambda.fun->formals->ellipsis && attrsUsed != arg.attrs->size()) - throwTypeError("function at %1% called with unexpected argument", fun.lambda.fun->pos); + argument (unless the attribute match specifies a `...'). */ + if (!fun.lambda.fun->formals->ellipsis && attrsUsed != arg.attrs->size()) { + /* Nope, so show the first unexpected argument to the + user. */ + foreach (Bindings::iterator, i, *arg.attrs) + if (fun.lambda.fun->formals->argNames.find(i->name) == fun.lambda.fun->formals->argNames.end()) + throwTypeError("function at %1% called with unexpected argument `%2%'", fun.lambda.fun->pos, i->name); + abort(); // can't happen + } } nrFunctionCalls++; |