Changeset 21656 for trunk/src/wsock32/asyncapi.cpp

Timestamp:

Jun 27, 2011, 5:14:44 PM (14 years ago)

Author:

dmik

Message:

wsock32: Don't start a dummy async thread when WSAEventSelect() is called to simply cancel a non-existing association on a socket. Since these calls are often made by applications before setting the socket to blocking mode even when no association on it has actually been created (per MSDN recommendations I suppose) and can occur at a frequent rate, this improvement may greatly reduce the number of async threads being created.

File:

• trunk/src/wsock32/asyncapi.cpp (modified) (1 diff)

trunk/src/wsock32/asyncapi.cpp

@@ -747 +747 @@
         return NO_ERROR;
    }
+   if(lEventMask == 0) {
+     // it's a call to cancel the non-existing association on this socket,
+     // return shortly (existing associations are canceled by the "if" above)
+     WSASetLastError(NO_ERROR);
+     return NO_ERROR;
+   }
+
    pThreadParm = (PASYNCTHREADPARM)malloc(sizeof(ASYNCTHREADPARM));
    if(pThreadParm == NULL) {