[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Changes by Serhiy Storchaka : -- nosy: +serhiy.storchaka ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Changes by Tal Einat : -- nosy: -taleinat ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Changes by Mark Lawrence : -- nosy: -BreamoreBoy ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Changes by Terry J. Reedy : -- versions: +Python 3.3, Python 3.4 -Python 3.1, Python 3.2 ___ Python tracker ___ ___ Python-bugs-list

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Changes by Andrew Svetlov : -- assignee: -> asvetlov nosy: +asvetlov ___ Python tracker ___ ___ Python-bugs-list mailing list Unsub

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Changes by Tal Einat : -- nosy: +taleinat ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Mark Lawrence added the comment: Given the current debate on python-dev regarding IDLE and its dependancy on tkinter, surely something as serious as a memory leak should be looked into. -- assignee: loewis -> nosy: +BreamoreBoy, tjreedy type: -> behavior versions: -Python 2.6 _

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Changes by Matthieu Labbé : -- nosy: +matthieu.labbe ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://m

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Changes by Terry J. Reedy : -- stage: -> patch review versions: +Python 3.2 -Python 2.5, Python 3.0 ___ Python tracker ___ ___ Pyth

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Changes by Guilherme Polo : Added file: http://bugs.python.org/file12430/keep_tclcommands_correct.diff ___ Python tracker ___ ___ Python-bug

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Changes by Guilherme Polo : Removed file: http://bugs.python.org/file12427/keep_tclcommands_correct.diff ___ Python tracker ___ ___ Python-b

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Guilherme Polo added the comment: I hope you are not too bored for me commenting on this again. So, I have re-though about this issue today and decided to solve it differently (and will include a patch here this time, don't worry about mentions to external repo this time). To solve the problem

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Guilherme Polo added the comment: On Wed, Dec 17, 2008 at 3:51 PM, Guilherme Polo wrote: > > Guilherme Polo added the comment: > > Rethinking about the changes done in Misc._configure I found out that > you don't need any of those there. > Isn't it the case of only improving the changes regard

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Guilherme Polo added the comment: I've fixed the leaks in Variable doing this: http://code.google.com/p/plumage/source/detail?r=93 I don't use an extra _tclCommands for Variable, instead in __del__ I use the information returned by Variable.trace_vinfo to remove associated callbacks. _

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Guilherme Polo added the comment: Rethinking about the changes done in Misc._configure I found out that you don't need any of those there. Isn't it the case of only improving the changes regarding callable overwriting in Misc._options ? So if the value is a callable and the option doesn't exist

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Guilherme Polo added the comment: Fix: My previous comment was about changes in Misc.deletecommand not Misc.destroy (this is what you get for guessing the methods changed in the diff, without applying it). To Quentin: I wouldn't bother mentioning it since this new TkinterError exception should

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Quentin Gallet-Gilles added the comment: A little remark : please replace "if not globals().has_key('TkinterError'):" by "if 'TkinterError' not in globals():". has_key is now deprecated in 2.6 and should be avoided. -- nosy: +quentin.gallet-gilles versions: -Python 2.5 ___

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Guilherme Polo added the comment: The changes in Misc.destroy do not look right, why are you deleting commands created by bind_all (for example) from the root window when a simple widget is destroyed ? It would make more sense to not apply these changes in Misc.destroy, instead, Misc.unbind_al

[issue1524639] Fix Tkinter Tcl-commands memory-leaks

Guilherme Polo added the comment: Something like the proposed patch is still needed. But allow me to point out my views towards your current patch: * Changes in Misc.after, Misc._bind: good * Changes in Misc.unbind can be simplified a bit: cbs = self._bind_names(self._bind(('bind', self._w),