PyObject_RichCompareBool() already returns -1, 0, or 1, so return its value

diff --git a/Objects/listobject.c b/Objects/listobject.c
index 229d64cfbd3bb3395ed545a199b671f6cfb6b305..64de38bcb326eb65866d08c020aad692b9163be5 100644 (file)
--- a/Objects/listobject.c
+++ b/Objects/listobject.c
@@ -325,11 +325,7 @@ list_contains(PyListObject *a, PyObject *el)
        for (i = 0, cmp = 0 ; cmp == 0 && i < a->ob_size; ++i)
                cmp = PyObject_RichCompareBool(el, PyList_GET_ITEM(a, i),
                                                   Py_EQ);
-       if (cmp > 0)
-               return 1;
-       if (cmp < 0)
-               return -1;
-       return 0;
+       return cmp;
 }
 
 

diff --git a/Objects/tupleobject.c b/Objects/tupleobject.c
index a6b862f442630c8ebcaea15f6e181e023918396b..6c2162aa691cc9098a8d0bcd030ef60fa5979ac9 100644 (file)
--- a/Objects/tupleobject.c
+++ b/Objects/tupleobject.c
@@ -266,11 +266,7 @@ tuplecontains(PyTupleObject *a, PyObject *el)
        for (i = 0, cmp = 0 ; cmp == 0 && i < a->ob_size; ++i)
                cmp = PyObject_RichCompareBool(el, PyTuple_GET_ITEM(a, i),
                                                   Py_EQ);
-       if (cmp > 0)
-               return 1;
-       if (cmp < 0)
-               return -1;
-       return 0;
+       return cmp;
 }
 
 static PyObject *