[issue30853] IDLE: configdialog -- factor out Tracer subclass

Terry J. Reedy report at bugs.python.org
Wed Jul 26 21:30:15 EDT 2017


Terry J. Reedy added the comment:

Go ahead.  If it works with font, add general tab.

CD.remove_var_callbacks is currently only called in test_configdialog.tearDownModule.  I added the call to prevent getting a TclError for each callback after the test finished.  .destroy does not destroy callbacks.

I presume remove_var_callbacks was written to be called in self.cancel. Then perhaps someone discovered that there was no visible effect of deleting it, perhaps because TclErrors are caught and ignored.  I want to put it, or the new equivalent, into cancel, before destroy.  And make  sure to add to teardownmodule.

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue30853>
_______________________________________


More information about the Python-bugs-list mailing list