From: Raymond Hettinger <rhettinger@users.noreply.github.com>
Date: Sun, 11 Nov 2018 22:35:47 +0000 (-0800)
Subject: Neaten the code without any algorithmic change. (GH-10466)
X-Git-Tag: v3.8.0a1~523
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f9ec1b9f5257b7a1cb6770e8398da626c86a5b7b;p=thirdparty%2FPython%2Fcpython.git

Neaten the code without any algorithmic change. (GH-10466)

Remove unneeded assertion (we already know so is a PySetObject *).
---

diff --git a/Objects/setobject.c b/Objects/setobject.c
index ce5092195975..035b1db06d8c 100644
--- a/Objects/setobject.c
+++ b/Objects/setobject.c
@@ -701,17 +701,14 @@ static PyObject *
 set_pop(PySetObject *so, PyObject *Py_UNUSED(ignored))
 {
     /* Make sure the search finger is in bounds */
-    setentry *entry, *limit;
+    setentry *entry = so->table + (so->finger & so->mask);
+    setentry *limit = so->table + so->mask;
     PyObject *key;
 
-    assert (PyAnySet_Check(so));
     if (so->used == 0) {
         PyErr_SetString(PyExc_KeyError, "pop from an empty set");
         return NULL;
     }
-
-    entry = so->table + (so->finger & so->mask);
-    limit = so->table + so->mask;
     while (entry->key == NULL || entry->key==dummy) {
         entry++;
         if (entry > limit)