Use connect_ex() instead of connect().

jeremyhylton · jeremyhylton · commit e16e54f7f1c0 · 2001-10-29T16:44:37.000Z
Removes old XXX comment and possible source of long-delays.
diff --git a/Lib/asyncore.py b/Lib/asyncore.py
@@ -53,7 +53,7 @@
 
 import os
 from errno import EALREADY, EINPROGRESS, EWOULDBLOCK, ECONNRESET, \
-     ENOTCONN, ESHUTDOWN, EINTR
+     ENOTCONN, ESHUTDOWN, EINTR, EISCONN
 
 try:
     socket_map
@@ -301,17 +301,14 @@ def bind (self, addr):
 
     def connect (self, address):
         self.connected = 0
-        # XXX why not use connect_ex?
-        try:
-            self.socket.connect (address)
-        except socket.error, why:
-            if why[0] in (EINPROGRESS, EALREADY, EWOULDBLOCK):
-                return
-            else:
-                raise socket.error, why
-        self.addr = address
-        self.connected = 1
-        self.handle_connect()
+        err = self.socket.connect_ex(address)
+        if err in (EINPROGRESS, EALREADY, EWOULDBLOCK):
+            return
+        if err in (0, EISCONN):
+            self.addr = address
+            self.connected = 1
+            self.handle_connect()
+        raise socket.error, err
 
     def accept (self):
         try:
