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

Interpreter - Remove checks for duplicate argument names

Browse source
This commit is contained in:
Emmanuel BENOîT 2023-01-02 21:49:45 +01:00
parent 29d3677b2a
commit c8be65dc4c
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/interpreter/functions.rs
View file

@ -47,8 +47,7 @@ impl Callable for Function {
assert_eq!(arguments.len(), self.arity()); assert_eq!(arguments.len(), self.arity());
let param_env = Environment::create_child(environment); let param_env = Environment::create_child(environment);
for (arg, value) in izip!(self.params.iter(), arguments.into_iter()) { for (arg, value) in izip!(self.params.iter(), arguments.into_iter()) {
// FIXME: duplicate parameter names should be detected in the parser param_env.borrow_mut().define(arg, Some(value)).unwrap();
param_env.borrow_mut().define(arg, Some(value))?;
} }
let child = Environment::create_child(&param_env); let child = Environment::create_child(&param_env);