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.

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

Implement zero-arg functions. Closes #5. #7

Merged
merged 2 commits into from
Oct 12, 2021
Merged

Conversation

namin
Copy link
Contributor

@namin namin commented Oct 11, 2021

The implementation simply desugars each zero-arg function into
a one-argument function taking nil.

The implementation simply desugars each zero-arg function into
a one-argument function taking nil.
@namin namin requested a review from porcuquine October 11, 2021 23:42
Copy link
Collaborator

@porcuquine porcuquine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. We can switch to the new continuation type as an optimization in the future, if we want. If you agree that's an option, can you just make a minimal issue to remind us of the possibility?

@namin
Copy link
Contributor Author

namin commented Oct 12, 2021

I added issue #8. I am not sure I understand why the continuation with a list used in lang.lisp has not been kept, since auto-currying also has an efficiency cost?

@porcuquine porcuquine mentioned this pull request Oct 12, 2021
@porcuquine porcuquine merged commit 67d2644 into master Oct 12, 2021
@porcuquine porcuquine deleted the thunk-as-sugar branch October 12, 2021 00:40
huitseeker pushed a commit that referenced this pull request Jun 15, 2023
Putting the build components in `rust-toolchain.toml` looks cleaner and
enhances compatibility with non-Nix users. This way, the only
maintenance going forward should be updating the sha256 hash with the
error message output if the toolchain is updated.

I also added the `nixpos-unstable` URL to have a common `nixpkgs`
channel between Nix users, and added the `ocl-icd` dependency for the
`groth16` feature.

Co-authored-by: John C. Burnham <jcb@johnchandlerburnham.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants