[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

STINNER Victor added the comment: Copy of interesting comments: https://github.com/python/cpython/pull/3025 serhiy-storchaka: "Parent's destroy() now is called even if this destroy() already was called. I.e. it can be called twice." haypo: "Yes, it's a deliberate choice. All other ttk widgets

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

STINNER Victor added the comment: New changeset 33460fa7e0bd126bee739a66e1228665dc22e70f by Victor Stinner in branch '3.6': ttk: fix LabeledScale and OptionMenu destroy() method (#3025) (#3030) https://github.com/python/cpython/commit/33460fa7e0bd126bee739a66e1228665dc22e70f -- _

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

Changes by STINNER Victor : -- pull_requests: +3065 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

STINNER Victor added the comment: New changeset cd7e9c1b67d3d07ee03e0a79af2791c19031cecb by Victor Stinner in branch 'master': ttk: fix LabeledScale and OptionMenu destroy() method (#3025) https://github.com/python/cpython/commit/cd7e9c1b67d3d07ee03e0a79af2791c19031cecb -- __

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

STINNER Victor added the comment: New changeset 56e162ad5c5d3effe9b4f05d0179e1b6a2a2d9b8 by Victor Stinner in branch '2.7': ttk: fix LabeledScale and OptionMenu destroy() method (#3026) https://github.com/python/cpython/commit/56e162ad5c5d3effe9b4f05d0179e1b6a2a2d9b8 -- _

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

STINNER Victor added the comment: widget_instance-2.7-v2.patch: Updated patch for Python 2.7 to debug test_ttk_guionly. https://github.com/python/cpython/pull/3026 fix LabeledScale and OptionMenu destroy() method for Python 2.7. -- Added file: http://bugs.python.org/file47064/widget_i

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

Changes by STINNER Victor : -- pull_requests: +3060 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

STINNER Victor added the comment: Example of failure on master using widget_instance-master.patch: haypo@selma$ ./python -m test -v -u all test_ttk_guionly == ERROR: test_initialization (tkinter.test.test_ttk.test_extensions.

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

Changes by STINNER Victor : -- pull_requests: +3059 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

STINNER Victor added the comment: widget_instance-master.patch: Check that Tkinter.Widget.destroy() removes all children and check that test_ttk_guionly doesn't leak widgets. -- Added file: http://bugs.python.org/file47063/widget_instance-master.patch __

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

STINNER Victor added the comment: On master, LabeledScaleTest.test_initialization() leaks a widget in the root widget, even after root.destroy(). The problem is that LabeledScaleTest.__del__() doesn't call Frame.destroy(self) if self._variable is not set. So master has at least this bug.

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

STINNER Victor added the comment: My widget_instance.patch calls test_support.gc_collect() and removes all widgets from the root window. It's not enough. That's why I proposed to modify LabeledScale.destroy(). -- ___ Python tracker

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

Serhiy Storchaka added the comment: See issue25130. Maybe forcing garbage collecting will help. -- ___ Python tracker ___ ___ Python-b

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

Changes by STINNER Victor : -- nosy: +gpolo ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

Changes by STINNER Victor : -- nosy: +serhiy.storchaka ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://

[issue31135] [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7

STINNER Victor added the comment: I only tested Python 2.7. I didn't check if other branches are affected. -- title: test_ttk_guionly doesn't destroy all widgets -> [2.7] test_ttk_guionly doesn't destroy all widgets on Python 2.7 ___ Python tracker