Sigh. That wasn't a memory leak, that was Guido committing before

running tests.  Withdraw 2.183 and its backport.

diff --git a/Objects/typeobject.c b/Objects/typeobject.c
index 2427c4e24ab7861e22cc9d345cb7db59a1842b74..8a1adc914594d5345d4567d12c15f8fdfc67d5d8 100644 (file)
--- a/Objects/typeobject.c
+++ b/Objects/typeobject.c
@@ -2249,11 +2249,8 @@ add_subclass(PyTypeObject *base, PyTypeObject *type)
        while (--i >= 0) {
                ref = PyList_GET_ITEM(list, i);
                assert(PyWeakref_CheckRef(ref));
-               if (PyWeakref_GET_OBJECT(ref) == Py_None) {
-                       i = PyList_SetItem(list, i, new);
-                       Py_DECREF(new);
-                       return i;
-               }
+               if (PyWeakref_GET_OBJECT(ref) == Py_None)
+                       return PyList_SetItem(list, i, new);
        }
        i = PyList_Append(list, new);
        Py_DECREF(new);