FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-44958: Fix ref. leak introduced in GH-27844 by erlend-aasland · Pull Request #28490 · python/cpython · GitHub

/ cpython Public

bpo-44958: Fix ref. leak introduced in GH-27844 - #28490

Merged
miss-islington merged 2 commits into
python:mainfrom
erlend-aasland:sqlite-fix-refleak
Sep 21, 2021
Merged

bpo-44958: Fix ref. leak introduced in GH-27844#28490
miss-islington merged 2 commits into
python:mainfrom
erlend-aasland:sqlite-fix-refleak

Conversation

erlend-aasland commented Sep 21, 2021
edited by miss-islington
Loading

Copy link
Copy Markdown
Contributor

Modify managed_connect() helper to support in-memory databases. Use it
for the regression tests added in GH-27844.

https://bugs.python.org/issue44958

Automerge-Triggered-By: GH:pablogsal

Erlend E. Aasland added 2 commits September 21, 2021 14:40
bedevere-bot added awaiting review tests Tests in the Lib/test dir labels Sep 21, 2021
erlend-aasland changed the title bpo-44958: Fix ref. leak in test_table_lock_cursor_non_readonly_select() bpo-44958: Fix ref. leak in test_table_lock_cursor_non_readonly_select Sep 21, 2021
erlend-aasland changed the title bpo-44958: Fix ref. leak in test_table_lock_cursor_non_readonly_select bpo-44958: Fix ref. leak introduced in GH-27844 Sep 21, 2021

Copy link
Copy Markdown
Contributor

@erlend-aasland: Status check is done, and it's a success ✅ .

miss-islington merged commit 3e3ff09 into python:main Sep 21, 2021
erlend-aasland deleted the sqlite-fix-refleak branch September 21, 2021 13:37
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL