[issue20080] Unused variable in Lib/sqlite3/test/factory.py

2014-02-15 Thread Roundup Robot
Roundup Robot added the comment: New changeset a87f284e14ea by Benjamin Peterson in branch '2.7': add missing test assertion (closes #20080) http://hg.python.org/cpython/rev/a87f284e14ea New changeset 66c7791d9841 by Benjamin Peterson in branch '3.3': add missing test assertion (closes #20080) h

[issue20080] Unused variable in Lib/sqlite3/test/factory.py

2014-02-15 Thread Ezio Melotti
Changes by Ezio Melotti : -- nosy: +ezio.melotti stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsu

[issue20080] Unused variable in Lib/sqlite3/test/factory.py

2014-01-01 Thread Vajrasky Kok
Vajrasky Kok added the comment: Thanks, Eric! Attached the patch to address Eric's concern. -- Added file: http://bugs.python.org/file33296/unused_variable_in_factory_py_v2.patch ___ Python tracker ___

[issue20080] Unused variable in Lib/sqlite3/test/factory.py

2013-12-27 Thread Eric V. Smith
Eric V. Smith added the comment: I think you want to either also testing the number of elements in t, or in just compare t to (row["a"], row["b"]) (untested). -- nosy: +eric.smith ___ Python tracker __

[issue20080] Unused variable in Lib/sqlite3/test/factory.py

2013-12-27 Thread Vajrasky Kok
New submission from Vajrasky Kok: There is unused variable t in Lib/sqlite3/test/factory.py. def CheckSqliteRowAsTuple(self): """Checks if the row object can be converted to a tuple""" self.con.row_factory = sqlite.Row row = self.con.execute("select 1 as a, 2 as b").f