Skip to content

bpo-40956: Convert sqlite3.connect and sqlite3.Connection.__init__ to AC #24421

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pablogsal merged 17 commits into from
Jun 20, 2021
Merged

bpo-40956: Convert sqlite3.connect and sqlite3.Connection.__init__ to AC #24421

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
8192606
Convert _sqlite3.connect() and _sqlite3.Connection.__init__() to Argu…
Jun 14, 2020
b72a29c
Add NEWS
Feb 2, 2021
d361aeb
Add test for 'check_same_thread' keyword
Mar 8, 2021
a861358
Add test for 'database' keyword
Mar 8, 2021
1ccf5e5
Merge branch 'main' into bpo-40956/finale
Jun 5, 2021
07164cb
Harden
Jun 5, 2021
a1e88c9
Simplify ref handling
Jun 5, 2021
4977d78
Use PyObject_CallFunction iso. PyObject_CallFunctionObjArgs in sqlite…
Jun 5, 2021
1b2720f
Adjust comment
Jun 5, 2021
c7cd9ee
Merge branch 'main' into bpo-40956/finale
Jun 6, 2021
39ed36b
Remove unused var from test_dont_check_same_thread
Jun 6, 2021
754fdc6
Resolve conflict: cached_statements now defaults to 128, not 100
Jun 13, 2021
e59af3b
Clinic
Jun 13, 2021
ad345e0
Merge branch 'main' into bpo-40956/finale
Jun 15, 2021
af0fc7e
Address review: wrap test_dont_check_same_thread with threading_helpe…
Jun 15, 2021
98f1a9d
Explain Py_DECREF of converted arguments
Jun 17, 2021
89926af
Merge branch 'main' into bpo-40956/finale
Jun 17, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Explain Py_DECREF of converted arguments
  • Loading branch information
Erlend E. Aasland committed Jun 17, 2021
commit 98f1a9d748e71c8e777dc51230d7f2866706a2c6
2 changes: 1 addition & 1 deletion Modules/_sqlite/connection.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ pysqlite_connection_init_impl(pysqlite_Connection *self,
(uri ? SQLITE_OPEN_URI : 0), NULL);
Py_END_ALLOW_THREADS

Py_DECREF(database_obj);
Py_DECREF(database_obj); // needed bco. the AC FSConverter

if (rc != SQLITE_OK) {
_pysqlite_seterror(self->db);
Expand Down
2 changes: 1 addition & 1 deletion Modules/_sqlite/module.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ pysqlite_connect_impl(PyObject *module, PyObject *database, double timeout,
timeout, detect_types,
isolation_level, check_same_thread,
factory, cached_statements, uri);
Py_DECREF(database);
Py_DECREF(database); // needed bco. the AC FSConverter
Py_DECREF(isolation_level);
return res;
}
Expand Down