diff -r 58e0d2c3ead8 Lib/tkinter/__init__.py --- a/Lib/tkinter/__init__.py Sun Aug 17 16:58:11 2014 +0300 +++ b/Lib/tkinter/__init__.py Sun Aug 17 19:39:59 2014 +0300 @@ -242,16 +242,9 @@ def get(self): """Return value of variable.""" return self._tk.globalgetvar(self._name) - def trace_variable(self, mode, callback): - """Define a trace callback for the variable. - - MODE is one of "r", "w", "u" for read, write, undefine. - CALLBACK must be a function which is called when - the variable is read, written or undefined. - - Return the name of the callback. - """ - f = CallWrapper(callback, None, self).__call__ + + def _register(self, callback): + f = CallWrapper(callback, None, self._root).__call__ cbname = repr(id(f)) try: callback = callback.__func__ @@ -265,6 +258,60 @@ if self._tclCommands is None: self._tclCommands = [] self._tclCommands.append(cbname) + return cbname + + def trace_add(self, mode, callback, *args): + """Define a trace callback for the variable. + + Mode is one of "array", "read", "write", "unset", or a list or tuple + of such strings. + Callback must be a function which is called when the variable is + accessed or modified via the array command, read, written or unset. + Additional parameters are given as parameters to the callback call. + + Return the name of the callback. + """ + cbname = self._register(callback) + self._tk.call('trace', 'add', 'variable', + self._name, mode, (cbname,) + args) + return cbname + + def trace_remove(self, mode, cbname, *args): + """Delete the trace callback for a variable. + + Mode is one of "array", "read", "write", "unset" or a list or tuple + of such strings. Should be same as were specified in trace_add(). + Cbname is the name of the callback returned from trace_add. + Additional parameters should be same as were specified in trace_add(). + """ + self._tk.call('trace', 'remove', 'variable', + self._name, mode, (cbname,) + args) + self._tk.deletecommand(cbname) + try: + self._tclCommands.remove(cbname) + except ValueError: + pass + + def trace_info(self): + """Return all trace callback information.""" + splitlist = self._tk.splitlist + return [(splitlist(k), splitlist(v)) for k, v in map(splitlist, + splitlist(self._tk.call('trace', 'info', 'variable', self._name)))] + + def trace_variable(self, mode, callback): + """Define a trace callback for the variable. + + MODE is one of "r", "w", "u" for read, write, undefine. + CALLBACK must be a function which is called when + the variable is read, written or undefined. + + Return the name of the callback. + + This method is deprecated and will likely be removed in a future + version of Tcl. Use trace_add() instead. + """ + # TODO: Add deprecation warning + cbname = self._register(callback) self._tk.call("trace", "variable", self._name, mode, cbname) return cbname trace = trace_variable @@ -273,7 +320,11 @@ MODE is one of "r", "w", "u" for read, write, undefine. CBNAME is the name of the callback returned from trace_variable or trace. + + This method is deprecated and will likely be removed in a future + version of Tcl. Use trace_remove() instead. """ + # TODO: Add deprecation warning self._tk.call("trace", "vdelete", self._name, mode, cbname) self._tk.deletecommand(cbname) try: @@ -281,7 +332,12 @@ except ValueError: pass def trace_vinfo(self): - """Return all trace callback information.""" + """Return all trace callback information. + + This method is deprecated and will likely be removed in a future + version of Tcl. Use trace_info() instead. + """ + # TODO: Add deprecation warning return [self._tk.split(x) for x in self._tk.splitlist( self._tk.call("trace", "vinfo", self._name))] def __eq__(self, other): diff -r 58e0d2c3ead8 Lib/tkinter/test/test_tkinter/test_variables.py --- a/Lib/tkinter/test/test_tkinter/test_variables.py Sun Aug 17 16:58:11 2014 +0300 +++ b/Lib/tkinter/test/test_tkinter/test_variables.py Sun Aug 17 19:39:59 2014 +0300 @@ -1,5 +1,5 @@ import unittest - +import gc from tkinter import Variable, StringVar, IntVar, DoubleVar, BooleanVar, Tk @@ -86,6 +86,80 @@ v.set("value") self.assertTrue(v.side_effect) + def test_trace_old(self): + v = Var() + vname = str(v) + trace = [] + def read_tracer(*args): + trace.append(('read',) + args) + def write_tracer(*args): + trace.append(('write',) + args) + cb1 = v.trace_variable('r', read_tracer) + cb2 = v.trace_variable('wu', write_tracer) + self.assertEqual(sorted(v.trace_vinfo()), [('r', cb1), ('wu', cb2)]) + self.assertEqual(trace, []) + + v.set('spam') + self.assertEqual(trace, [('write', vname, '', 'w')]) + + trace = [] + v.get() + self.assertEqual(trace, [('read', vname, '', 'r')]) + + trace = [] + v.trace_vdelete('r', cb1) + self.assertEqual(v.trace_vinfo(), [('wu', cb2)]) + v.get() + self.assertEqual(trace, []) + + trace = [] + del write_tracer + v.set('eggs') + self.assertEqual(trace, [('write', vname, '', 'w')]) + + trace = [] + del v + gc.collect() + self.assertEqual(trace, [('write', vname, '', 'u')]) + + def test_trace(self): + v = Var() + vname = str(v) + trace = [] + def read_tracer(*args): + trace.append(('read',) + args) + def write_tracer(*args): + trace.append(('write',) + args) + cb1 = v.trace_add('read', read_tracer, 42) + cb2 = v.trace_add(['write', 'unset'], write_tracer) + self.assertEqual(sorted(v.trace_info()), [ + (('read',), (cb1, '42')), + (('write', 'unset'), (cb2,))]) + self.assertEqual(trace, []) + + v.set('spam') + self.assertEqual(trace, [('write', vname, '', 'write')]) + + trace = [] + v.get() + self.assertEqual(trace, [('read', '42', vname, '', 'read')]) + + trace = [] + v.trace_remove('read', cb1, 42) + self.assertEqual(v.trace_info(), [(('write', 'unset'), (cb2,))]) + v.get() + self.assertEqual(trace, []) + + trace = [] + del write_tracer + v.set('eggs') + self.assertEqual(trace, [('write', vname, '', 'write')]) + + trace = [] + del v + gc.collect() + self.assertEqual(trace, [('write', vname, '', 'unset')]) + class TestStringVar(TestBase):