Skip to content

[2.7] bpo-32857: Raise error when tkinter after_cancel() is called with None (GH-5701)#6620

Merged
serhiy-storchaka merged 5 commits into
python:2.7from
csabella:backport-74382a3-2.7
May 5, 2018
Merged

[2.7] bpo-32857: Raise error when tkinter after_cancel() is called with None (GH-5701)#6620
serhiy-storchaka merged 5 commits into
python:2.7from
csabella:backport-74382a3-2.7

Conversation

@csabella

@csabella csabella commented Apr 27, 2018

Copy link
Copy Markdown
Contributor

(cherry picked from commit 74382a3)

https://bugs.python.org/issue32857

@csabella

Copy link
Copy Markdown
Contributor Author

@serhiy-storchaka I had to change the side effect on the tests because nonlocal was invalid. Modifying a dictionary was suggested on SO, but if that isn't the best solution, then I can change it.

@serhiy-storchaka serhiy-storchaka 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.

Thank you @csabella!

I usually use a list, but using a dict is a good solution too, and it it may be the best solution in this case.

@serhiy-storchaka serhiy-storchaka merged commit 3a04598 into python:2.7 May 5, 2018
@serhiy-storchaka

Copy link
Copy Markdown
Member

Finally I have caught the moment when tests are passed and the branch is in sync! 😫

@csabella csabella deleted the backport-74382a3-2.7 branch May 6, 2018 12:57
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.

4 participants