tseeker-pub/rust-crafting-interpreters-part-1
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Interpreter - Check arity before calling

Browse source
This commit is contained in:
Emmanuel BENOîT 2023-01-02 17:47:53 +01:00
parent d47adab0f8
commit 3750b5226b
1 changed files with 14 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/interpreter/interpretable.rs
View file

@ -398,8 +398,20 @@ impl ast::ExprNode {
}
v
};
if let Value::Callable(callable) = &callee {
Ok(callable.borrow().call(environment, arg_values)?.into())
if let Value::Callable(callable_ref) = &callee {
let callable = callable_ref.borrow();
if callable.arity() != arg_values.len() {
Err(InterpreterError::new(
right_paren,
&format!(
"expected {} arguments, found {}",
arg_values.len(),
callable.arity()
),
))
} else {
Ok(callable.call(environment, arg_values)?.into())
}
} else {
Err(InterpreterError::new(
right_paren,