Add a name binding pass for variables

sophiajt / new-nu-parser

23 stars 10 forks source link

Add a name binding pass for variables #18

Closed kubouch closed 8 months ago

kubouch commented 9 months ago

Resolution pass performs binding of definition and use sites. Only implemented for variables in this PR. Includes:

Scoping logic (entering and leaving blocks)
- includes an idea how overlays could work, but overlays are not implemented yet
Variable name resolution:
- let and closure params define variables within scope
- $var performs variable lookup within the current scope frame
Added the scope information into the test report
Added errors to the test report

kubouch commented 9 months ago

I want to make sure the test outputs make sense and add scoping to more commands (if, for, etc.). Commands like for should also define variables using the same trick as I did in the closure resolving.