summaryrefslogtreecommitdiffstats
path: root/generic
diff options
context:
space:
mode:
authordgp <dgp@users.sourceforge.net>2007-09-07 15:51:24 (GMT)
committerdgp <dgp@users.sourceforge.net>2007-09-07 15:51:24 (GMT)
commit0fff7f052b013ecc6e4194b8568bc7c9e1a9c4f6 (patch)
tree1004390eda2e52d8425094b14c5693604de37d1f /generic
parent7a1583786a195fc3b4d1c4e95a223a2113f700c2 (diff)
downloadtcl-0fff7f052b013ecc6e4194b8568bc7c9e1a9c4f6.zip
tcl-0fff7f052b013ecc6e4194b8568bc7c9e1a9c4f6.tar.gz
tcl-0fff7f052b013ecc6e4194b8568bc7c9e1a9c4f6.tar.bz2
* generic/tclResult.c (Tcl_GetReturnOptions): Take care that a
* tests/init.test: non-TCL_ERROR code doesn't cause existing -errorinfo, -errorcode, and -errorline entries to be omitted. * generic/tclEvent.c: With -errorInfo no longer lost, generate more complete ::errorInfo when calling [bgerror] after a non-TCL_ERROR background exception.
Diffstat (limited to 'generic')
-rw-r--r--generic/tclEvent.c46
-rw-r--r--generic/tclResult.c13
2 files changed, 26 insertions, 33 deletions
diff --git a/generic/tclEvent.c b/generic/tclEvent.c
index 4b37b1e..78b44c2 100644
--- a/generic/tclEvent.c
+++ b/generic/tclEvent.c
@@ -12,7 +12,7 @@
* See the file "license.terms" for information on usage and redistribution of
* this file, and for a DISCLAIMER OF ALL WARRANTIES.
*
- * RCS: @(#) $Id: tclEvent.c,v 1.74 2007/09/06 18:13:19 dgp Exp $
+ * RCS: @(#) $Id: tclEvent.c,v 1.75 2007/09/07 15:51:25 dgp Exp $
*/
#include "tclInt.h"
@@ -356,35 +356,29 @@ TclDefaultBgErrorHandlerObjCmd(
tempObjv[1] = Tcl_ObjPrintf("command returned bad code: %d", code);
break;
}
- if (code == TCL_ERROR) {
- /*
- * Restore important state variables to what they were at the time
- * the error occurred.
- *
- * Need to set the variables, not the interp fields, because
- * Tcl_EvalObjv calls Tcl_ResetResult which would destroy
- * anything we write to the interp fields.
- */
+ Tcl_IncrRefCount(tempObjv[1]);
- TclNewLiteralStringObj(keyPtr, "-errorcode");
- Tcl_IncrRefCount(keyPtr);
- Tcl_DictObjGet(NULL, objv[2], keyPtr, &valuePtr);
- Tcl_DecrRefCount(keyPtr);
- if (valuePtr) {
- Tcl_SetVar2Ex(interp, "errorCode", NULL, valuePtr, TCL_GLOBAL_ONLY);
- }
+ TclNewLiteralStringObj(keyPtr, "-errorcode");
+ Tcl_IncrRefCount(keyPtr);
+ Tcl_DictObjGet(NULL, objv[2], keyPtr, &valuePtr);
+ Tcl_DecrRefCount(keyPtr);
+ if (valuePtr) {
+ Tcl_SetObjErrorCode(interp, valuePtr);
+ }
- TclNewLiteralStringObj(keyPtr, "-errorinfo");
- Tcl_IncrRefCount(keyPtr);
- Tcl_DictObjGet(NULL, objv[2], keyPtr, &valuePtr);
- Tcl_DecrRefCount(keyPtr);
- if (valuePtr) {
- Tcl_SetVar2Ex(interp, "errorInfo", NULL, valuePtr, TCL_GLOBAL_ONLY);
+ TclNewLiteralStringObj(keyPtr, "-errorinfo");
+ Tcl_IncrRefCount(keyPtr);
+ Tcl_DictObjGet(NULL, objv[2], keyPtr, &valuePtr);
+ Tcl_DecrRefCount(keyPtr);
+ if (valuePtr) {
+ if (code != TCL_ERROR) {
+ Tcl_SetObjResult(interp, tempObjv[1]);
}
- } else {
- Tcl_AppendObjToErrorInfo(interp, Tcl_DuplicateObj(tempObjv[1]));
+ Tcl_IncrRefCount(valuePtr);
+ Tcl_AppendObjToErrorInfo(interp, valuePtr);
}
- Tcl_IncrRefCount(tempObjv[1]);
+
+ /* Capture stack trace now, so we can report it if [bgerror] fails. */
valuePtr = Tcl_GetVar2Ex(interp, "errorInfo", NULL, TCL_GLOBAL_ONLY);
Tcl_IncrRefCount(valuePtr);
diff --git a/generic/tclResult.c b/generic/tclResult.c
index c77e634..64be014 100644
--- a/generic/tclResult.c
+++ b/generic/tclResult.c
@@ -8,7 +8,7 @@
* See the file "license.terms" for information on usage and redistribution of
* this file, and for a DISCLAIMER OF ALL WARRANTIES.
*
- * RCS: @(#) $Id: tclResult.c,v 1.37 2007/06/05 17:57:08 dgp Exp $
+ * RCS: @(#) $Id: tclResult.c,v 1.38 2007/09/07 15:51:26 dgp Exp $
*/
#include "tclInt.h"
@@ -1452,14 +1452,13 @@ Tcl_GetReturnOptions(
}
if (result == TCL_ERROR) {
- /*
- * When result was an error, fill in any missing values for
- * -errorinfo, -errorcode, and -errorline.
- */
-
Tcl_AddObjErrorInfo(interp, "", -1);
- Tcl_DictObjPut(NULL, options, keys[KEY_ERRORINFO], iPtr->errorInfo);
+ }
+ if (iPtr->errorCode) {
Tcl_DictObjPut(NULL, options, keys[KEY_ERRORCODE], iPtr->errorCode);
+ }
+ if (iPtr->errorInfo) {
+ Tcl_DictObjPut(NULL, options, keys[KEY_ERRORINFO], iPtr->errorInfo);
Tcl_DictObjPut(NULL, options, keys[KEY_ERRORLINE],
Tcl_NewIntObj(iPtr->errorLine));
}