Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Interpret let-values #7

Closed
pmatos opened this issue Mar 12, 2023 · 1 comment · Fixed by #33
Closed

Interpret let-values #7

pmatos opened this issue Mar 12, 2023 · 1 comment · Fixed by #33
Labels
enhancement New feature or request interpreter

Comments

@pmatos
Copy link
Owner

pmatos commented Mar 12, 2023

No description provided.

@pmatos pmatos added enhancement New feature or request interpreter labels Mar 12, 2023
@chatcody
Copy link

chatcody bot commented Mar 12, 2023

As an AI language model, I cannot provide a solution without further information on the issue. Kindly provide more details on the problem with "Interpret let-values" and how it relates to "null". With more information, I can provide a solution and implementation in code if necessary.

pmatos added a commit that referenced this issue Apr 8, 2023
@pmatos
Implement let-values
6b4e93d 
This is a rather large change because to implement let-values, we fixed a
couple of things along the way.

* `define-values` should not create a new environment. Just adds to the
current one.
* as a consequence of the previous point, we need to add an empty env
at interpreter construction time.
* implement a downcast function from expr to value as that is needed around a
few places.
* add more implementations for dumper.

Fixes #7
@pmatos pmatos mentioned this issue Apr 8, 2023
Merged
@pmatos pmatos closed this as completed in #33 Apr 8, 2023
pmatos added a commit that referenced this issue Apr 8, 2023
@pmatos
Implement let-values (#33)
d99153f 
This is a rather large change because to implement let-values, we fixed a
couple of things along the way.

* `define-values` should not create a new environment. Just adds to the
current one.
* as a consequence of the previous point, we need to add an empty env
at interpreter construction time.
* implement a downcast function from expr to value as that is needed around a
few places.
* add more implementations for dumper.

Fixes #7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request interpreter
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement let-values pmatos/nora
1 participant
@pmatos