Bugs item #3394168, was opened at 2011-08-19 00:34
Message generated for change (Settings changed) made by bagder
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=3394168&group_id=976

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: libcurl
Group: new feature request
>Status: Closed
Resolution: Invalid
Priority: 5
Private: No
Submitted By: Mark A. Hershberger (hexmode)
Assigned to: Daniel Stenberg (bagder)
Summary: please apply hiphop patch

Initial Comment:
Here it is: https://raw.github.com/facebook/hiphop-php/master/src/third_party/libcurl.fb-changes.diff Also attached.

----------------------------------------------------------------------

Comment By: Daniel Stenberg (bagder)
Date: 2011-08-19 09:33

Message:
Thanks,

If FD_SETSIZE is a problem you're using a very large amount of file
descriptors. If you're using a large amount of file descriptors then poll()
is also very slow and a bad choice. Hence libcurl provides the multi_socket
API to allow the app to work around both the FD_SETSIZE problem as well as
completely avoiding slow functions such as poll() and select().

I'm not seeing the major benefit of this function.

If you still think this is good stuff for libcurl, then I would like to
see that you motivate and explain your patch and reasoning on the
curl-library list so that we can discuss pros and cons and see what others
think and how to proceed. (The patch also lacks documentation and test
cases.)

This is not a bug, so this bug entry will be closed.

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=3394168&group_id=976
Received on 2011-09-06