Skip to content

bpo-40959: Remove unused and unneeded declarations from sqlite3 headers #20828

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

Jump to bottom
Merged
merged 1 commit into from
Jan 6, 2021
Merged

bpo-40959: Remove unused and unneeded declarations from sqlite3 headers #20828

merged 1 commit into from
Jan 6, 2021

Conversation

erlend-aasland
Copy link
Contributor

@erlend-aasland erlend-aasland commented Jun 12, 2020 •
edited by bedevere-bot
Loading

I guess skipping NEWS is ok for this one.

https://bugs.python.org/issue40959

@erlend-aasland
Copy link
Contributor Author

@berkerpeksag If you feel I'm flooding you with PR's/code reviewing, please let me know. Also, I might have missed a point or two in the devs guide, so if I'm proceeding with PR's/issues in a totally incorrect matter, feel free to point me in the right direction.

@erlend-aasland
Copy link
Contributor Author

FYI, rebased onto master

@erlend-aasland
Copy link
Contributor Author

@corona10 would you mind reviewing this? It's a straight-forward cleanup of the _sqlite3 header files:

  • remove function declarations for functions that are not used by other .c files
  • remove function declarations for functions that don't exist anymore

Diffstat:

 Modules/_sqlite/cache.h            |  5 -----
 Modules/_sqlite/connection.h       |  7 -------
 Modules/_sqlite/cursor.h           | 10 ----------
 Modules/_sqlite/prepare_protocol.h |  3 ---
 Modules/_sqlite/statement.h        |  1 -
 5 files changed, 26 deletions(-)

berkerpeksag
berkerpeksag approved these changes Jan 6, 2021
View reviewed changes
Copy link
Member

@berkerpeksag berkerpeksag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Could you please fix merge conflicts?

@erlend-aasland
Copy link
Contributor Author

LGTM! Could you please fix merge conflicts?

Sure. 2 sec...

berkerpeksag
berkerpeksag approved these changes Jan 6, 2021
View reviewed changes
Copy link
Member

@berkerpeksag berkerpeksag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@berkerpeksag berkerpeksag merged commit 203b249 into python:master Jan 6, 2021
@erlend-aasland erlend-aasland mentioned this pull request Jan 6, 2021
Closed
@erlend-aasland erlend-aasland mentioned this pull request Feb 19, 2021
Merged
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 13, 2021
@adorilson
bpo-40959: Remove unused declarations from sqlite3 headers (pythonGH-…
93b6d03 
…20828)
@erlend-aasland erlend-aasland deleted the fix-issue-40959 branch August 22, 2023 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@berkerpeksag berkerpeksag berkerpeksag approved these changes

Assignees
No one assigned
Labels
skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@erlend-aasland @berkerpeksag @the-knights-who-say-ni @bedevere-bot