[issue21682] Refleak in idle_test test_autocomplete

2014-06-08 Thread Terry J. Reedy
Terry J. Reedy added the comment: Checking the buildbot just now, there is some other stuff after test_tk on X86 Windows 7, but I ran the command you gave (but with python_d) and it passed ok. So I presume this is really fixed and should stay closed. -- ___

[issue21682] Refleak in idle_test test_autocomplete

2014-06-08 Thread Roundup Robot
Roundup Robot added the comment: New changeset b8f33440cd5e by Terry Jan Reedy in branch '2.7': Issue #21682: Replace EditorWindow with mock to eliminate memory leaks. http://hg.python.org/cpython/rev/b8f33440cd5e New changeset e6cc02d32957 by Terry Jan Reedy in branch '3.4': Issue #21682: Repla

[issue21682] Refleak in idle_test test_autocomplete

2014-06-08 Thread Zachary Ware
Zachary Ware added the comment: Terry, did you mean to push Saimadhav's patch? -- ___ Python tracker ___ ___ Python-bugs-list mailing

[issue21682] Refleak in idle_test test_autocomplete

2014-06-07 Thread Terry J. Reedy
Terry J. Reedy added the comment: This concerns me. I expect that we will eventually want to test a live EditorWindow or subclass. It appears that root.destroy does not clear all the widgets created by EditorWindow(root=root). My guess it that something is created without passing in root, so t

[issue21682] Refleak in idle_test test_autocomplete

2014-06-06 Thread Saimadhav Heblikar
Saimadhav Heblikar added the comment: The patch fixes the refleak. Importing EditorWindow, was perhaps the cause. It uses a dummy editwin instead. With reference to the current test, was there a particular reason to import real EditorWindow module? -- keywords: +patch Added file: htt

[issue21682] Refleak in idle_test test_autocomplete

2014-06-06 Thread Zachary Ware
New submission from Zachary Ware: The recently added test_autocomplete seems to be hanging onto a reference somewhere that it shouldn't be, see below. This output was obtained by running `python -m test -R :: -uall test_idle`. I tracked it down to test_autocomplete with hg bisect, and proved