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

Improve folds #264

Merged
merged 4 commits into from
Sep 10, 2018
Merged

Improve folds #264

merged 4 commits into from
Sep 10, 2018

Conversation

basil-conto
Copy link
Collaborator

Please review each commit individually. The key points are:

  • Make recursive folds iterative to avoid exceeding max-lisp-eval-depth.
  • Handle an empty list argument properly in -reductions and -reductions-r.

The docs will probably need to be regenerated if any other commits are added in the meantime.

@basil-conto
Copy link
Collaborator Author

Friendly ping.

@Fuco1 Fuco1 self-assigned this Apr 21, 2018
* Translate recursion to iteration to avoid exceeding
  max-lisp-eval-depth.

* Rewrite functions in terms of anaphoric macros.

* Expand test cases.
@basil-conto
Copy link
Collaborator Author

I rebased and resolved merge conflicts, FWIW. Anything else holding this up?

@Fuco1 Fuco1 merged commit 6514359 into magnars:master Sep 10, 2018
@Fuco1
Copy link
Collaborator

Fuco1 commented Sep 10, 2018

It took some time but we've arrived! Thanks!

@basil-conto basil-conto deleted the blc/reduce branch September 11, 2018 01:00
@basil-conto
Copy link
Collaborator Author

Thanks! :) I think create-docs.sh has to be re-run following #263, though.

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.

2 participants