Skip to content

gh-108278: Deprecate passing the three first params as keyword args for sqlite3 UDF creation APIs #108281

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
erlend-aasland merged 10 commits into from
Aug 28, 2023
Merged

gh-108278: Deprecate passing the three first params as keyword args for sqlite3 UDF creation APIs #108281

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
4f89b4c
gh-108278: Clean up some sqlite3 connection APIs
erlend-aasland Aug 22, 2023
634217b
Sphinx fixup
erlend-aasland Aug 22, 2023
7fa3edb
Pull in main
erlend-aasland Aug 23, 2023
cedd33a
Restrict PR to only touch the UDF creation APIs
erlend-aasland Aug 23, 2023
54e35d5
Update docs
erlend-aasland Aug 23, 2023
833197a
Add test
erlend-aasland Aug 23, 2023
661cd7a
Update Doc/whatsnew/3.13.rst
erlend-aasland Aug 23, 2023
f72c5e9
Pull in main
erlend-aasland Aug 27, 2023
37205c4
Pull in main
erlend-aasland Aug 28, 2023
15cf809
Address review: inline tests
erlend-aasland Aug 28, 2023
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
Restrict PR to only touch the UDF creation APIs
  • Loading branch information
@erlend-aasland
erlend-aasland committed Aug 23, 2023
commit cedd33a0d6a402708925a5388c0528c60ba3692c
7 changes: 0 additions & 7 deletions Doc/whatsnew/3.13.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -261,13 +261,6 @@ Deprecated

The affected parameters will become positional-only in Python 3.15.

Deprecate passing the callback as a keyword argument, for the following
:class:`sqlite3.Connection` APIs:

* :meth:`~sqlite3.Connection.set_authorizer`
* :meth:`~sqlite3.Connection.set_progress_handler`
* :meth:`~sqlite3.Connection.set_trace_callback`

The affected parameters will become positional-only in Python 3.15.
Comment thread
erlend-aasland marked this conversation as resolved.
Outdated
(Contributed by Erlend E. Aasland in :gh:`107948` and :gh:`108278`.)

Expand Down
7 changes: 0 additions & 7 deletions Misc/NEWS.d/next/Library/2023-08-22-13-51-10.gh-issue-108278.11d_qG.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,6 @@ arguments, for the following :class:`sqlite3.Connection` APIs:
* :meth:`~sqlite3.Connection.create_function`
* :meth:`~sqlite3.Connection.create_aggregate`

Deprecate passing the callback as a keyword argument, for the following
:class:`sqlite3.Connection` APIs:

* :meth:`~sqlite3.Connection.set_authorizer`
* :meth:`~sqlite3.Connection.set_progress_handler`
* :meth:`~sqlite3.Connection.set_trace_callback`

The affected parameters will become positional-only in Python 3.15.

Patch by Erlend E. Aasland.
86 changes: 4 additions & 82 deletions Modules/_sqlite/clinic/connection.c.h
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 3 additions & 6 deletions Modules/_sqlite/connection.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1501,7 +1501,6 @@ _sqlite3.Connection.set_authorizer as pysqlite_connection_set_authorizer
cls: defining_class
/
authorizer_callback as callable: object
/ [from 3.15]

Sets authorizer callback.
[clinic start generated code]*/
Expand All @@ -1510,7 +1509,7 @@ static PyObject *
pysqlite_connection_set_authorizer_impl(pysqlite_Connection *self,
PyTypeObject *cls,
PyObject *callable)
/*[clinic end generated code: output=75fa60114fc971c3 input=d4bf5013974daf98]*/
/*[clinic end generated code: output=75fa60114fc971c3 input=605d32ba92dd3eca]*/
{
if (!pysqlite_check_thread(self) || !pysqlite_check_connection(self)) {
return NULL;
Expand Down Expand Up @@ -1544,7 +1543,6 @@ _sqlite3.Connection.set_progress_handler as pysqlite_connection_set_progress_han
cls: defining_class
/
progress_handler as callable: object
/ [from 3.15]
n: int

Sets progress handler callback.
Expand All @@ -1554,7 +1552,7 @@ static PyObject *
pysqlite_connection_set_progress_handler_impl(pysqlite_Connection *self,
PyTypeObject *cls,
PyObject *callable, int n)
/*[clinic end generated code: output=0739957fd8034a50 input=9c8776d84cefd42b]*/
/*[clinic end generated code: output=0739957fd8034a50 input=f7c1837984bd86db]*/
{
if (!pysqlite_check_thread(self) || !pysqlite_check_connection(self)) {
return NULL;
Expand Down Expand Up @@ -1582,7 +1580,6 @@ _sqlite3.Connection.set_trace_callback as pysqlite_connection_set_trace_callback
cls: defining_class
/
trace_callback as callable: object
/ [from 3.15]

Sets a trace callback called for each SQL statement (passed as unicode).
[clinic start generated code]*/
Expand All @@ -1591,7 +1588,7 @@ static PyObject *
pysqlite_connection_set_trace_callback_impl(pysqlite_Connection *self,
PyTypeObject *cls,
PyObject *callable)
/*[clinic end generated code: output=d91048c03bfcee05 input=69944bec571b97d2]*/
/*[clinic end generated code: output=d91048c03bfcee05 input=351a94210c5f81bb]*/
{
if (!pysqlite_check_thread(self) || !pysqlite_check_connection(self)) {
return NULL;
Expand Down