Message361006
pysqlite_cursor_fetchall currently has the following bit of code:
/* 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);
}
}
This can and should be rewritten as a for loop to avoid the unnecessary initialization to Py_None and the redundant if statement inside the loop.
pysqlite_cursor_fetchmany has the same problem. |
|
Date |
User |
Action |
Args |
2020-01-30 03:18:40 | alex.henrie | set | recipients:
+ alex.henrie |
2020-01-30 03:18:40 | alex.henrie | set | messageid: <1580354320.44.0.87062873054.issue39496@roundup.psfhosted.org> |
2020-01-30 03:18:40 | alex.henrie | link | issue39496 messages |
2020-01-30 03:18:40 | alex.henrie | create | |
|