This is the mail archive of the insight@sources.redhat.com mailing list for the Insight project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH] return errors from varobj_get_value


Hi,

Ever notice that errors in the variable windows are not printed in the
window? That's because we never put them there!

This patch fixes variable_value so that when an error occurs, the error
string gets shoved into tcl so that the variable windows will use it.

Keith

ChangeLog
2002-01-14  Keith Seitz  <keiths@redhat.com>

	* generic/gdbtk-varobj.c (variable_value): When varobj_get_value
	fails, use error_last_message to return the error to tcl land.
	This allows us to display nice messages about why "0xdeadbeef" is
	not accessible!

Patch
Index: generic/gdbtk-varobj.c
===================================================================
RCS file: /cvs/src/src/gdb/gdbtk/generic/gdbtk-varobj.c,v
retrieving revision 1.10
diff -p -r1.10 gdbtk-varobj.c
*** generic/gdbtk-varobj.c	2001/11/05 19:42:48	1.10
--- generic/gdbtk-varobj.c	2002/01/15 02:14:51
*************** variable_value (interp, objc, objv, var)
*** 609,615 ****
    r = varobj_get_value (var);

    if (r == NULL)
!     return TCL_ERROR;
    else
      {
        Tcl_SetObjResult (interp, Tcl_NewStringObj (r, -1));
--- 609,620 ----
    r = varobj_get_value (var);

    if (r == NULL)
!     {
!       char *err = error_last_message ();
!       gdbtk_set_result (interp, "%s", err);
!       xfree (err);
!       return TCL_ERROR;
!     }
    else
      {
        Tcl_SetObjResult (interp, Tcl_NewStringObj (r, -1));


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]