diff options
author | dkf <donal.k.fellows@manchester.ac.uk> | 2005-06-14 13:45:57 (GMT) |
---|---|---|
committer | dkf <donal.k.fellows@manchester.ac.uk> | 2005-06-14 13:45:57 (GMT) |
commit | 9480f190e5c0b49511894e60db3c34b9a7a81003 (patch) | |
tree | 8753974f9a48a1a793cb9bd53171a42b17f86238 /generic/tclTrace.c | |
parent | cdd600c79b33458ca7ca7861563f1c6756e69ff6 (diff) | |
download | tcl-9480f190e5c0b49511894e60db3c34b9a7a81003.zip tcl-9480f190e5c0b49511894e60db3c34b9a7a81003.tar.gz tcl-9480f190e5c0b49511894e60db3c34b9a7a81003.tar.bz2 |
Fix [Bug 1220058] and quash a bizarre case which generated a bogus error msg.
Thanks to Will Duquette for helping to track this one down.
Diffstat (limited to 'generic/tclTrace.c')
-rw-r--r-- | generic/tclTrace.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/generic/tclTrace.c b/generic/tclTrace.c index 3a15ac8..d482a6b 100644 --- a/generic/tclTrace.c +++ b/generic/tclTrace.c @@ -11,7 +11,7 @@ * See the file "license.terms" for information on usage and redistribution * of this file, and for a DISCLAIMER OF ALL WARRANTIES. * - * RCS: @(#) $Id: tclTrace.c,v 1.23 2005/05/10 18:34:51 kennykb Exp $ + * RCS: @(#) $Id: tclTrace.c,v 1.24 2005/06/14 13:46:03 dkf Exp $ */ #include "tclInt.h" @@ -1284,6 +1284,7 @@ TraceCommandProc(clientData, interp, oldName, newName, flags) */ if (flags & (TCL_TRACE_DESTROYED | TCL_TRACE_DELETE)) { int untraceFlags = tcmdPtr->flags; + Tcl_InterpState state; if (tcmdPtr->stepTrace != NULL) { Tcl_DeleteTrace(interp, tcmdPtr->stepTrace); @@ -1315,9 +1316,14 @@ TraceCommandProc(clientData, interp, oldName, newName, flags) * Remove the trace since TCL_TRACE_DESTROYED tells us to, or the * command we're tracing has just gone away. Then decrement the * clientData refCount that was set up by trace creation. + * + * Note that we save the (return) state of the interpreter to + * prevent bizarre error messages. */ + state = Tcl_SaveInterpState(interp, TCL_OK); Tcl_UntraceCommand(interp, oldName, untraceFlags, TraceCommandProc, clientData); + (void) Tcl_RestoreInterpState(interp, state); tcmdPtr->refCount--; } if ((--tcmdPtr->refCount) <= 0) { |