Change error return value to be more consistent with the rest of Python
authorBrett Cannon <brett@python.org>
Wed, 7 Sep 2016 19:51:08 +0000 (12:51 -0700)
committerBrett Cannon <brett@python.org>
Wed, 7 Sep 2016 19:51:08 +0000 (12:51 -0700)
Objects/codeobject.c

index c8abda202b2135f7456a11bcf8e45b0ad2f8e839..5da2e93ab265a05d1bd1524f3d72bfd237d595ae 100644 (file)
@@ -780,7 +780,7 @@ _PyCode_GetExtra(PyObject *code, Py_ssize_t index, void **extra)
 
     if (!PyCode_Check(code)) {
         PyErr_BadInternalCall();
-        return 1;
+        return -1;
     }
 
     o = (PyCodeObject*) code;
@@ -803,7 +803,7 @@ _PyCode_SetExtra(PyObject *code, Py_ssize_t index, void *extra)
     if (!PyCode_Check(code) || index < 0 ||
             index >= tstate->co_extra_user_count) {
         PyErr_BadInternalCall();
-        return 1;
+        return -1;
     }
 
     o = (PyCodeObject*) code;
@@ -812,13 +812,13 @@ _PyCode_SetExtra(PyObject *code, Py_ssize_t index, void *extra)
         o->co_extra = (_PyCodeObjectExtra*) PyMem_Malloc(
             sizeof(_PyCodeObjectExtra));
         if (o->co_extra == NULL) {
-            return 1;
+            return -1;
         }
 
         o->co_extra->ce_extras = PyMem_Malloc(
             tstate->co_extra_user_count * sizeof(void*));
         if (o->co_extra->ce_extras == NULL) {
-            return 1;
+            return -1;
         }
 
         o->co_extra->ce_size = tstate->co_extra_user_count;
@@ -832,7 +832,7 @@ _PyCode_SetExtra(PyObject *code, Py_ssize_t index, void *extra)
             o->co_extra->ce_extras, tstate->co_extra_user_count * sizeof(void*));
 
         if (o->co_extra->ce_extras == NULL) {
-            return 1;
+            return -1;
         }
 
         o->co_extra->ce_size = tstate->co_extra_user_count;