Skip to content

bpo-34794: Fix a leak in Tkinter. #10025

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.

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

Merged

Conversation

serhiy-storchaka
Copy link
Member

@serhiy-storchaka serhiy-storchaka commented Oct 21, 2018

@vstinner
Copy link
Member

@serhiy-storchaka: Hum, it seems like this fix can be merged, but it seems like you prefer to merge your own PRs :-)

@serhiy-storchaka
Copy link
Member Author

Thank you @vstinner and @zhangyangyu.

@serhiy-storchaka
Copy link
Member Author

Actually it is @zhangyangyu have found the bug.

@serhiy-storchaka serhiy-storchaka merged commit df13df4 into python:master Oct 23, 2018
@miss-islington
Copy link
Contributor

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

@serhiy-storchaka serhiy-storchaka deleted the tkinter-tcl_obj-leak branch October 23, 2018 19:40
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 23, 2018
Based on the investigation by Xiang Zhang.
(cherry picked from commit df13df4)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 23, 2018
Based on the investigation by Xiang Zhang.
(cherry picked from commit df13df4)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-bot
Copy link

GH-10069 is a backport of this pull request to the 3.7 branch.

@bedevere-bot
Copy link

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

@miss-islington
Copy link
Contributor

Sorry, @serhiy-storchaka, I could not cleanly backport this to 2.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker df13df41a25765d8a39a77220691698498e758d4 2.7

miss-islington added a commit that referenced this pull request Oct 23, 2018
Based on the investigation by Xiang Zhang.
(cherry picked from commit df13df4)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
vstinner pushed a commit that referenced this pull request Oct 23, 2018
Based on the investigation by Xiang Zhang.
(cherry picked from commit df13df4)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this pull request Oct 28, 2018
Based on the investigation by Xiang Zhang..
(cherry picked from commit df13df4)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-bot
Copy link

GH-10181 is a backport of this pull request to the 2.7 branch.

serhiy-storchaka added a commit that referenced this pull request Oct 28, 2018
Based on the investigation by Xiang Zhang.
(cherry picked from commit df13df4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants