After some careful review, this compiler warning can be done away with

  on win32, since it serves no purpose on this platform.


git-svn-id: http://svn.apache.org/repos/asf/apr/apr/trunk@61740 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 5 insertions, 1 deletions

diff --git a/network_io/win32/sockets.c b/network_io/win32/sockets.c
index dff1cd780..5a567eb27 100644
--- a/network_io/win32/sockets.c
+++ b/network_io/win32/sockets.c
@@ -290,7 +290,11 @@ APR_DECLARE(apr_status_t) apr_connect(apr_socket_t *sock, apr_sockaddr_t *sa)
         /* wait for the connect to complete */
         FD_ZERO(&temp);
         FD_SET(sock->sock, &temp);
-        if (select(sock->sock+1, NULL, &temp, NULL, NULL) == SOCKET_ERROR) {
+        /* the select(nfds, ...) nfds arg is ignored 
+         * we don't have a bit table for fd_set on Win32,
+         * we have a messy dynamic crossref table.
+         */
+        if (select(FD_SETSIZE+1, NULL, &temp, NULL, NULL) == SOCKET_ERROR) {
             return apr_get_netos_error();
         }
     }