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

Can we implement a faster version of alongside? #43

Closed
ekmett opened this issue Sep 10, 2012 · 1 comment
Closed

Can we implement a faster version of alongside? #43

ekmett opened this issue Sep 10, 2012 · 1 comment

Comments

@ekmett
Copy link
Owner

ekmett commented Sep 10, 2012

We switched to a final encoding of Bazaar because it made cloning dramatically faster. Would a final encoding of Context have a similar performance impact? e.g.

newtype Store c d a = Store { runStore :: forall f. Functor f => (c -> f d) -> f a }

This might also speed up a number of Control.Lens.Plated combinators.

@ekmett
Copy link
Owner Author

ekmett commented Sep 11, 2012

Benchmarks seem to bear out that the current version is the fastest reasonable implementation. Closing this out.

@ekmett ekmett closed this as completed Sep 11, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant