Skip to content

Conversation

@serhiy-storchaka
Copy link
Member

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

Copy link
Member

@vstinner vstinner left a comment

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
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

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