Issue #16704: Get rid of select.error in stdlib. Use OSError instead.

diff --git a/Lib/subprocess.py b/Lib/subprocess.py
index 09c85da4c0fbfaa378fdae924c0a0e05cc85a06f..ac960f9ad54b99eef50a0caae4c14dd3e704570d 100644 (file)
--- a/Lib/subprocess.py
+++ b/Lib/subprocess.py
@@ -1614,7 +1614,7 @@ class Popen(object):
                     raise TimeoutExpired(self.args, orig_timeout)
                 try:
                     ready = poller.poll(timeout)
-                except select.error as e:
+                except OSError as e:
                     if e.args[0] == errno.EINTR:
                         continue
                     raise
@@ -1682,7 +1682,7 @@ class Popen(object):
                     (rlist, wlist, xlist) = \
                         select.select(self._read_set, self._write_set, [],
                                       timeout)
-                except select.error as e:
+                except OSError as e:
                     if e.args[0] == errno.EINTR:
                         continue
                     raise

diff --git a/Lib/telnetlib.py b/Lib/telnetlib.py
index a59693e746315de8babd832f8c42c91d124fcf62..b7c57da2092d6b23a8a656e372301f457c352bef 100644 (file)
--- a/Lib/telnetlib.py
+++ b/Lib/telnetlib.py
@@ -313,7 +313,7 @@ class Telnet:
             while i < 0 and not self.eof:
                 try:
                     ready = poller.poll(call_timeout)
-                except select.error as e:
+                except OSError as e:
                     if e.errno == errno.EINTR:
                         if timeout is not None:
                             elapsed = time() - time_start
@@ -683,7 +683,7 @@ class Telnet:
             while not m and not self.eof:
                 try:
                     ready = poller.poll(call_timeout)
-                except select.error as e:
+                except OSError as e:
                     if e.errno == errno.EINTR:
                         if timeout is not None:
                             elapsed = time() - time_start

diff --git a/Lib/test/test_logging.py b/Lib/test/test_logging.py
index 54ab4862dde3299f3a74f38cb9e71487e0f4bbb0..9b760555b42c5aeb38e733b1e5bc9ff7dc7fab54 100644 (file)
--- a/Lib/test/test_logging.py
+++ b/Lib/test/test_logging.py
@@ -773,7 +773,7 @@ if threading:
             """
             try:
                 asyncore.loop(poll_interval, map=self.sockmap)
-            except select.error:
+            except OSError:
                 # On FreeBSD 8, closing the server repeatably
                 # raises this error. We swallow it if the
                 # server has been closed.

diff --git a/Lib/test/test_select.py b/Lib/test/test_select.py
index dee59210756f55d2582bd6a55129dd31e0fcde51..8f9a1c9d88eab55b28f0c9b1e6e325bebcda26b9 100644 (file)
--- a/Lib/test/test_select.py
+++ b/Lib/test/test_select.py
@@ -32,7 +32,7 @@ class SelectTestCase(unittest.TestCase):
             fp.close()
             try:
                 select.select([fd], [], [], 0)
-            except select.error as err:
+            except OSError as err:
                 self.assertEqual(err.errno, errno.EBADF)
             else:
                 self.fail("exception not raised")

diff --git a/Lib/test/test_signal.py b/Lib/test/test_signal.py
index c973be8b6ece388a14ee2d825dc7c1ff39b53a05..97e8d223342c34366988108232117db57f08e210 100644 (file)
--- a/Lib/test/test_signal.py
+++ b/Lib/test/test_signal.py
@@ -299,10 +299,10 @@ class WakeupSignalTests(unittest.TestCase):
             # We attempt to get a signal during the select call
             try:
                 select.select([read], [], [], TIMEOUT_FULL)
-            except select.error:
+            except OSError:
                 pass
             else:
-                raise Exception("select.error not raised")
+                raise Exception("OSError not raised")
             after_time = time.time()
             dt = after_time - before_time
             if dt >= TIMEOUT_HALF: