Refactor Env - Githubissues

spy16 / parens

Parens is a highly flexible and embeddable LISP toolkit. :computer:

33 stars 3 forks source link

Refactor Env #27

Closed lthibault closed 4 years ago

lthibault commented 4 years ago

Fixes issues flagged in #26:

Export Analyze() from Env
Bind Env to Expr

⏱️ Estimated review time: 10 minutes ✅ Merge when ready.

lthibault commented 4 years ago

@spy16 Should Invokable still take an env argument? It seems like it should since it might be invoked in a different scope from where it was defined, but I'm not sure.

spy16 commented 4 years ago

Yea, Invokable would still need to take env for the same reason you mentioned.