Bugs item #1365447, was opened at 2005-11-24 11:35
Message generated for change (Comment added) made by bagder
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=1365447&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: Open
Resolution: None
>Priority: 2
Submitted By: Olaf van der Spek (olafvdspek)
Assigned to: Daniel Stenberg (bagder)
Summary: curl_multi_fdset: max_fd will contain -1

Initial Comment:
Hi,

man select

> n is the highest-numbered descriptor in any of the
three sets, plus 1.

http://curl.haxx.se/libcurl/c/curl_multi_fdset.html

> If no file descriptors are set by libcurl, max_fd
will contain -1 when this function returns.

Shouldn't this be 0 instead of -1?
And wouldn't it be better to use max_fd as in in/out
param so it can be easier used with multiple calls to
curl_multi_fdset?

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

>Comment By: Daniel Stenberg (bagder)
Date: 2005-11-24 12:27

Message:
Logged In: YES
user_id=1110

Perhaps, but this is the API as it works today and it seems
like a very minor change that would have a serious impact on
all existing applications using this function.

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

Comment By: Olaf van der Spek (olafvdspek)
Date: 2005-11-24 11:57

Message:
Logged In: YES
user_id=30264

Nevermind about the max_fd issue, you're supposed to add 1
before you call select.
The out vs in/out issue still stands.

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

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=1365447&group_id=976
Received on 2005-11-24