Lita Cho added the comment:

I tested the patch and it looks correct upon inspection. It looks like it 
applies cleanly and a straight forward solution. 

I made a slight change so that when the Demo exits, it has a better message. I 
also added the change to the NEWS.txt file. 

I also ran the IDLE tests, and everything passed. I didn't see new tests added, 
but I am not sure if that is needed, since we are just adding a menu binding.

I've attached my patch with the changes listed above. It should be ready for 
review.

----------
nosy: +Lita.Cho, jesstess
Added file: http://bugs.python.org/file35328/issue.patch

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue17172>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to