Simplify groupby re-entrancy test

Author: VanshAgarwal24036

Lib/test/test_itertools.py

@@ -742,9 +742,7 @@ def __eq__(self, other):
                     next(g)
                 return False
 
-        data = [Key(b"a"), Key(b"b")]
-
-        g = itertools.groupby(data)
+        g = itertools.groupby([Key(b"a"), Key(b"b")])
         next(g)
         next(g)  # must not segfault
 

Modules/itertoolsmodule.c

@@ -544,7 +544,6 @@ groupby_next(PyObject *op)
         else if (gbo->tgtkey == NULL)
             break;
         else {
-            int rcmp;
 
             /* A user-defined __eq__ can re-enter groupby and advance the iterator,
                mutating gbo->tgtkey / gbo->currkey while we are comparing them.
@@ -556,7 +555,7 @@ groupby_next(PyObject *op)
             Py_INCREF(tgtkey);
             Py_INCREF(currkey);
 
-            rcmp = PyObject_RichCompareBool(tgtkey, currkey, Py_EQ);
+            int rcmp = PyObject_RichCompareBool(tgtkey, currkey, Py_EQ);
 
             Py_DECREF(tgtkey);
             Py_DECREF(currkey);