[dgorbachev] 43303: select: change it to be more readable, avoid a spurious compiler warning.

Author: dgorbachev
Date: Mon Oct  5 16:13:15 2009
New Revision: 43303

URL: http://svn.reactos.org/svn/reactos?rev=43303&view=rev
Log:
select: change it to be more readable, avoid a spurious compiler warning.

Modified:
    trunk/reactos/dll/win32/ws2_32_new/src/select.c

Modified: trunk/reactos/dll/win32/ws2_32_new/src/select.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/ws2_32_new/src/se...
==============================================================================
--- trunk/reactos/dll/win32/ws2_32_new/src/select.c [iso-8859-1] (original)
+++ trunk/reactos/dll/win32/ws2_32_new/src/select.c [iso-8859-1] Mon Oct  5 16:13:15 2009
@@ -48,72 +48,69 @@
     PWSSOCKET Socket;
     INT Status;
     INT ErrorCode;
-    BOOL Found = FALSE;
     SOCKET Handle;
+    LPWSPSELECT WSPSelect;
+
     DPRINT("select: %lx %p %p %p %p\n", s, readfds, writefds, exceptfds, timeout);

-#if defined(__GNUC__) && \
-    (__GNUC__ * 100 + __GNUC_MINOR__ == 404)
-    /* Silence incorrect GCC 4.4.x warning */
-    Handle = 0;
-#endif
-
     /* Check for WSAStartup */
-    if ((ErrorCode = WsQuickProlog()) == ERROR_SUCCESS)
-    {
-        /* Use the first Socket from the first valid set */
-        if (readfds && readfds->fd_count) 
-        {
-            Handle = readfds->fd_array[0];
-            Found = TRUE;
-        } 
-        if (!Found && writefds && writefds->fd_count) 
-        {
-            Handle = writefds->fd_array[0];
-            Found = TRUE;
-        } 
-        if (!Found && exceptfds && exceptfds->fd_count) 
-        {
-            Handle = exceptfds->fd_array[0];
-            Found = TRUE;
-        }
-
-        /* Make sure we found one */
-        if (Found)
-        {
-            /* Get the Socket Context */
-            if ((Socket = WsSockGetSocket(Handle)))
-            {
-                /* Make the call */
-                Status = Socket->Provider->Service.lpWSPSelect(s,
-                                                               readfds, 
-                                                               writefds,
-                                                               exceptfds, 
-                                                               (struct timeval *)timeout, 
-                                                               &ErrorCode);
-                /* Deference the Socket Context */
-                WsSockDereference(Socket);
-
-                /* Return Provider Value */
-                if (Status != SOCKET_ERROR) return Status;
-
-                /* If everything seemed fine, then the WSP call failed itself */
-                if (ErrorCode == NO_ERROR) ErrorCode = WSASYSCALLFAILURE;
-            }
-            else
-            {
-                /* No Socket Context Found */
-                ErrorCode = WSAENOTSOCK;
-            }
-        }
-        else
-        {
-            /* Invalid handles */
-            ErrorCode = WSAEINVAL;
-        }
-    }
-
-    /* Return with an Error */
+    ErrorCode = WsQuickProlog();
+
+    if (ErrorCode != ERROR_SUCCESS)
+    {
+        SetLastError(ErrorCode);
+        return SOCKET_ERROR;
+    }
+
+    /* Use the first Socket from the first valid set */
+    if (readfds && readfds->fd_count)
+    {
+        Handle = readfds->fd_array[0];
+    }
+    else if (writefds && writefds->fd_count)
+    {
+        Handle = writefds->fd_array[0];
+    }
+    else if (exceptfds && exceptfds->fd_count)
+    {
+        Handle = exceptfds->fd_array[0];
+    }
+    else
+    {
+        /* Invalid handles */
+        SetLastError(WSAEINVAL);
+        return SOCKET_ERROR;
+    }
+
+    /* Get the Socket Context */
+    Socket = WsSockGetSocket(Handle);
+
+    if (!Socket)
+    {
+        /* No Socket Context Found */
+        SetLastError(WSAENOTSOCK);
+        return SOCKET_ERROR;
+    }
+
+    /* Get the select procedure */
+    WSPSelect = Socket->Provider->Service.lpWSPSelect;
+
+    /* Make the call */
+    Status = WSPSelect(s, readfds, writefds, exceptfds, (struct timeval *)timeout,
+                       &ErrorCode);
+
+    /* Deference the Socket Context */
+    WsSockDereference(Socket);
+
+    /* Return Provider Value */
+    if (Status != SOCKET_ERROR)
+        return Status;
+
+    /* If everything seemed fine, then the WSP call failed itself */
+    if (ErrorCode == NO_ERROR)
+        ErrorCode = WSASYSCALLFAILURE;
+
+    /* Return with an error */
     SetLastError(ErrorCode);
     return SOCKET_ERROR;
 }