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

An example of curl_multi #74

Closed
wants to merge 1 commit into from
Closed

Conversation

posix4e
Copy link

@posix4e posix4e commented Jun 11, 2015

I'm looking at adding support for the curl_multi api. This creates a lot of duplication (which I have tried to avoid) and most likely we will need a different library that pulls in eventual and mio to actually make use of the bindings. I'm working on one called micro-hyper. Consider this pull a demonstration and a WIP. The question I have is what the api should look like.

@carllerche
Copy link
Contributor

Sorry for the delay. Trying to catch up.

I definitely want to get curl multi bound. Glancing through the PR though, it looks like there is a lot of duplication. Any way to try to avoid duplicating definitions?

@posix4e
Copy link
Author

posix4e commented Aug 4, 2015

No worries. Notice CurlM vs Curl for a lot of the options. What dups are you most worried about?

@carllerche
Copy link
Contributor

It's a bit hard for me to follow (and I have lost a lot of context), but I'm noticing some duplication like CURL_VERSION_IPV6.

@posix4e
Copy link
Author

posix4e commented Aug 19, 2015

So i removed the easy.rs file altogether. Is this any clearer?

@alexcrichton
Copy link
Owner

Ok, I've started binding the multi interface in #109, so I'm gonna close this now. Sorry for the delayed response :)

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.

3 participants