homepage

In the _loadtk method of the class Tk, there are two
calls self.tk.createcommand, one for 'tkerror' and one
for 'exit'. In the createcommand function in
_tkinter.c, there is a Py_XINCREF for the Tk widget and
one for the Python function (_tkerror or _exit). The
corresponding Py_XDECREFs are called (via
Tkapp_DeleteCommand) in PythonCmdDelete. The
Tkapp_DeleteCommand is called by the deletecommand in
the destroy method of class Misc in Tkinter.py.
However, the deletecommand is called only for commands
listed in self._tclCommands. Now, if a command is
created via the _register method in class Misc, the
command is appended to self._tclCommands. However, the
'tkerror' and 'exit' commands, created in the _loadtk
method of the class Tk, are not appended to
self._tclCommands. Hence, the '_tkerror' and '_exit'
functions as well as the Tk widget itself have an
incorrect reference count. The Tkapp_Dealloc routine in
_tkinter.c is not called and the Tk widget is not
destroyed.
The attached patch simply adds a
self._tclCommands.append for 'tkerror' and 'exit'. With
this patch, a Tk widget is destroyed and Tkapp_Dealloc
is called correctly.

Logged In: YES 
user_id=21627
Wouldn't it be better to use _register for _tkerror and
_exit as well? This patch overwrites self._tclCommands, but
the object may already have a _tclCommands attribute.

Logged In: YES 
user_id=488897
If we use _register without modification, we'll end up with
a different command name:
>>> from Tkinter import Label
>>> l = Label(text="some text")
+ Tkinter created command -1214096332_tkerror
+ Tkinter created command -1208510068_exit
+ Tkinter created command -1208511348destroy
>>>
whereas the command names should be "_tkerror", "_exit".
We can modify the _register functon so that it takes an
optional "name" keyword argument. However, it will mean that
we create the command through the CallWrapper instead of the
command directly. For "_exit", if I understand the code
correctly, it would mean that the raise in _exit is obscured
by the raise in CallWrapper, which doesn't seem the right thing.
So I would suggest to add a "if self._tclCommands==None:" to
the patch instead of going through _register. I'll update
the patch if you agree.

Logged In: YES 
user_id=21627
Thanks for the patch, applied as
Tkinter.py 1.182 and 1.181.2.1
NEWS 1.1248 and 1.1193年2月23日
I added the test for _tclCommands is None, please verify the
code.
BTW, how would I make the exit command work? It seems that
it does not
properly cause a Python exit, but instead causes a TclError.

Logged In: YES 
user_id=488897
I tried the CVS version on a few platforms (Cygwin, Mac OS
X, Linux) and I found no problems with it. Also the Python
test suite didn't reveal any problems. Thanks for looking at
this patch. I really appreciate your 5-to-1 patch review offer.
To answer your question about the exit command, it may be an
attempt to replace Tcl's exit command, so that the Tcl
interpreter doesn't exit. But I don't know, really, I don't
use Tkinter much. My real interest is the PyOS_InputHook
event hook function, which I need to work correctly for a
scientific plotting package for Python. While working on a
separate patch involving PyOS_InputHook, I noticed that
Tkinter doesn't call the DisableEventHook problem. This led
me to the reference count problem.