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

Optimize sized.ToHList and ToSizedHList #1182

Merged
merged 1 commit into from
May 31, 2021

Conversation

joroKr21
Copy link
Collaborator

  • Use type parameters and Aux instead of path-dependent types
  • Optimize runtime by replacing direct recursion with foldRight

Fixes #1181

 * Use type parameters and `Aux` instead of path-dependent types
 * Optimize runtime by replacing direct recursion with `foldRight`
@joroKr21 joroKr21 marked this pull request as ready for review May 31, 2021 17:45
@joroKr21 joroKr21 requested a review from milessabin May 31, 2021 17:45
@joroKr21 joroKr21 self-assigned this May 31, 2021
Copy link
Owner

@milessabin milessabin left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants