Skip to content

bpo-32365: Fix a reference leak when compile __debug__.#4916

Merged
serhiy-storchaka merged 1 commit into
python:masterfrom
serhiy-storchaka:compile-debug-name-leak
Dec 18, 2017
Merged

bpo-32365: Fix a reference leak when compile __debug__.#4916
serhiy-storchaka merged 1 commit into
python:masterfrom
serhiy-storchaka:compile-debug-name-leak

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Dec 18, 2017

Copy link
Copy Markdown
Member

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I didn't check if the change is correct, but I confirm that it fix the reference leak using "./python -m test -R 3:3 test_ast test_builtin test_compile".

Note: I had to disable PtyTests in test_builtin, but it's unrelated to this PR.

@serhiy-storchaka serhiy-storchaka merged commit bd6ec4d into python:master Dec 18, 2017
@serhiy-storchaka serhiy-storchaka deleted the compile-debug-name-leak branch December 18, 2017 12:29
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.6.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 18, 2017
It was introduced in bpo-27169.
(cherry picked from commit bd6ec4d)
@bedevere-bot

Copy link
Copy Markdown

GH-4918 is a backport of this pull request to the 3.6 branch.

serhiy-storchaka pushed a commit that referenced this pull request Dec 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants