[Python-checkins] bpo-39496: Remove redundant checks from _sqlite/cursor.c (GH-18270)

Alex Henrie webhook-mailer at python.org
Sat Feb 1 15:45:48 EST 2020


https://github.com/python/cpython/commit/78c7183f470b60a39ac2dd0ad1a94d49d1e0b062
commit: 78c7183f470b60a39ac2dd0ad1a94d49d1e0b062
branch: master
author: Alex Henrie <alexhenrie24 at gmail.com>
committer: GitHub <noreply at github.com>
date: 2020-02-01T23:45:34+03:00
summary:

bpo-39496: Remove redundant checks from _sqlite/cursor.c (GH-18270)

files:
M Modules/_sqlite/cursor.c

diff --git a/Modules/_sqlite/cursor.c b/Modules/_sqlite/cursor.c
index 2302ca9edac2d..06275ecb26849 100644
--- a/Modules/_sqlite/cursor.c
+++ b/Modules/_sqlite/cursor.c
@@ -786,17 +786,9 @@ PyObject* pysqlite_cursor_fetchmany(pysqlite_Cursor* self, PyObject* args, PyObj
         return NULL;
     }
 
-    /* just make sure we enter the loop */
-    row = Py_None;
-
-    while (row) {
-        row = pysqlite_cursor_iternext(self);
-        if (row) {
-            PyList_Append(list, row);
-            Py_DECREF(row);
-        } else {
-            break;
-        }
+    while ((row = pysqlite_cursor_iternext(self))) {
+        PyList_Append(list, row);
+        Py_XDECREF(row);
 
         if (++counter == maxrows) {
             break;
@@ -821,15 +813,9 @@ PyObject* pysqlite_cursor_fetchall(pysqlite_Cursor* self, PyObject* args)
         return NULL;
     }
 
-    /* just make sure we enter the loop */
-    row = (PyObject*)Py_None;
-
-    while (row) {
-        row = pysqlite_cursor_iternext(self);
-        if (row) {
-            PyList_Append(list, row);
-            Py_DECREF(row);
-        }
+    while ((row = pysqlite_cursor_iternext(self))) {
+        PyList_Append(list, row);
+        Py_XDECREF(row);
     }
 
     if (PyErr_Occurred()) {



More information about the Python-checkins mailing list