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.
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
gh-93421: Update sqlite3 cursor.rowcount after SQLITE_DONE #93526
Conversation
If you want to schedule another build, you need to add the " |
FYI, the failing buildbot are also failing on |
If you want to schedule another build, you need to add the " |
I'm going to merge this in a day or two. The buildbot failures are unrelated to this PR. |
-{"rowcount", T_LONG, offsetof(pysqlite_Cursor, rowcount), READONLY},
+{"rowcount", T_LONGLONG, offsetof(pysqlite_Cursor, rowcount), READONLY},
|
This comment was marked as off-topic.
This comment was marked as off-topic.
Good catch!
I'm fine with backporting that, but I'll remove it from this PR nonetheless. It makes the PR cleaner.
Yes, we could do a lot of refactoring in the whole sqlite3 module. I prefer to keep refactorings in separate PRs. Thanks for reviewing! |
If you want to schedule another build, you need to add the " |
FTR, buildbot failures are unrelated to this PR. |
Thanks @erlend-aasland for the PR |
GH-93598 is a backport of this pull request to the 3.11 branch. |
Sorry, @erlend-aasland, I could not cleanly backport this to |
…pythonGH-93526) (cherry picked from commit 875de61) Co-authored-by: Erlend Egeberg Aasland <erlend.aasland@protonmail.com>
…TE_DONE (pythonGH-93526) (cherry picked from commit 875de61)
GH-93599 is a backport of this pull request to the 3.10 branch. |
Alternative to #93520
Resolves #93421