]> granicus.if.org Git - python/commitdiff
Fix a probable merge glitch in r66695:
authorAmaury Forgeot d'Arc <amauryfa@gmail.com>
Tue, 30 Sep 2008 20:22:44 +0000 (20:22 +0000)
committerAmaury Forgeot d'Arc <amauryfa@gmail.com>
Tue, 30 Sep 2008 20:22:44 +0000 (20:22 +0000)
a redundant check that actually belongs to another function.

Modules/_stringio.c

index d29fe64eba70393ba261de4d352bfb1ade90ace1..f342048d5daf5f3621c5656c699a7b5960010da1 100644 (file)
@@ -140,6 +140,8 @@ stringio_read(StringIOObject *self, PyObject *args)
 
     if (PyLong_Check(arg)) {
         size = PyLong_AsSsize_t(arg);
+        if (size == -1 && PyErr_Occurred())
+            return NULL;
     }
     else if (arg == Py_None) {
         /* Read until EOF is reached, by default. */
@@ -179,8 +181,6 @@ stringio_truncate(StringIOObject *self, PyObject *args)
         size = PyLong_AsSsize_t(arg);
         if (size == -1 && PyErr_Occurred())
             return NULL;
-        if (size == -1 && PyErr_Occurred())
-            return NULL;
     }
     else if (arg == Py_None) {
         /* Truncate to current position if no argument is passed. */