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

Removal of curry2 and curry2flipped #41

Merged
merged 1 commit into from May 30, 2013
Merged

Conversation

eborden
Copy link
Contributor

@eborden eborden commented May 30, 2013

These methods become redundant with the introduction of curry and rCurry.

fogus added a commit that referenced this pull request May 30, 2013
Removal of curry2 and curry2flipped
@fogus fogus merged commit 5d80fba into documentcloud:master May 30, 2013
@raganwald
Copy link
Contributor

My allong.es library has both curry2 and generic curry functions because it does a lot of binary currying itself, and curry2 is faster than curry. But I can see the argument for keeping the API simpler.

@fogus
Copy link
Member

fogus commented May 30, 2013

I think a case can be made for fast versions of 2 and 3-arg currying. I can't personally recall a time when I've curried more than 3. I'd be open to putting curry2, rcurry2, curry3 and rcurry3 (back) in. Maybe another ticket is in order?

@eborden
Copy link
Contributor Author

eborden commented May 31, 2013

I'd be curious to see a perf of these functions. I'm sure curry2 is more performant, but the proof is in the pudding

@fogus
Copy link
Member

fogus commented May 31, 2013

Moved discussion to #42

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

3 participants