bpo-38962: Fix reference leak in new_interpreter() by pablogsal · Pull Request #17453 · python/cpython
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 34.2k
Merged
miss-islington merged 2 commits intopython:masterfrom
Dec 4, 2019Merged
bpo-38962: Fix reference leak in new_interpreter()#17453
miss-islington merged 2 commits intopython:masterfrom
bpo-38962: Fix reference leak in new_interpreter()#17453
miss-islington merged 2 commits intopython:masterfrom
Conversation
pablogsal
requested a review
from vstinner
the-knights-who-say-ni
added
the
CLA signed
label
bedevere-bot
added
the
awaiting core review
label
pablogsal
added
the
skip news
label
pablogsal
force-pushed
the
bpo-38962
branch
2 times, most recently
from
f13e0a7 to
64e9705
Compare
pablogsal
force-pushed
the
bpo-38962
branch
from
64e9705 to
2006275
Compare
vstinner reviewed Dec 4, 2019
Python/pylifecycle.c
Outdated
Show resolved
Hide resolved
Python/pylifecycle.c Outdated Show resolved Hide resolved
pablogsal
self-assigned this
pablogsal
added
the
🤖 automerge
label
vstinner approved these changes Dec 4, 2019
Copy link
Member
vstinner
left a comment
vstinner
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
bedevere-bot
added
awaiting merge
and removed
awaiting core review
labels
miss-islington
merged commit
b96c6b0
into
python:master
bedevere-bot
removed
the
awaiting merge
label
pablogsal
deleted the
bpo-38962
branch
jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request
Dec 5, 2019shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request
Jan 31, 2020This 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