1999-01-29 James Ingham <jingham@cygnus.com>
authorJim Ingham <jingham@apple.com>
Sat, 30 Jan 1999 04:20:31 +0000 (04:20 +0000)
committerJim Ingham <jingham@apple.com>
Sat, 30 Jan 1999 04:20:31 +0000 (04:20 +0000)
* gdbtk.c (gdbtk_init): Set the fputs_unfiltered_hook to
gdbtk_fputs BEFORE you eval script.  The old code was setting it
to null until after you did this, but that is wrong, because it
will cause the output of CAUGHT errors to go to gdb_stderr, which
is wrong.  You only want to write errors to the console if the
eval generates an error.

gdb/ChangeLog-gdbtk
gdb/gdbtk.c

index 9bfc14942fa8f04013ee6f8ebbfec09c718fa18e..d3de7207abc10ad9f3eacd91862918fc67eb2bbb 100644 (file)
@@ -1,3 +1,12 @@
+1999-01-29  James Ingham  <jingham@cygnus.com>
+
+       * gdbtk.c (gdbtk_init): Set the fputs_unfiltered_hook to
+       gdbtk_fputs BEFORE you eval script.  The old code was setting it
+       to null until after you did this, but that is wrong, because it
+       will cause the output of CAUGHT errors to go to gdb_stderr, which
+       is wrong.  You only want to write errors to the console if the
+       eval generates an error.
+
 1999-01-29  Martin Hunt  <hunt@cygnus.com>
 
        * gdbtk-cmds.c (gdb_get_breakpoint_info): When printing addresses,
index 61075fbf72cb745ee9e43b85d70d453b25125c64..3e96505798fe535aa50169966877ff70a3c8f38a 100644 (file)
@@ -558,7 +558,7 @@ proc gdbtk_find_main {} {\n\
 gdbtk_find_main";
 #endif /* NO_TCLPRO_DEBUGGER */
     
-    fputs_unfiltered_hook = NULL; /* Force errors to stdout/stderr */
+    /* fputs_unfiltered_hook = NULL; */ /* Force errors to stdout/stderr */
     
     /*
      * Set the variables for external editor, do this before eval'ing main.tcl
@@ -570,7 +570,9 @@ gdbtk_find_main";
     Tcl_SetVar (gdbtk_interp, "external_editor_command",
                external_editor_command, 0);
     
-    if (Tcl_GlobalEval (gdbtk_interp, (char *) script) != TCL_OK)
+  fputs_unfiltered_hook = gdbtk_fputs;
+
+  if (Tcl_GlobalEval (gdbtk_interp, (char *) script) != TCL_OK)
       {
        char *msg;
        
@@ -592,10 +594,6 @@ gdbtk_find_main";
       }
   }
 
-/* Defer setup of fputs_unfiltered_hook to near the end so that error messages
-   prior to this point go to stdout/stderr.  */
-
-  fputs_unfiltered_hook = gdbtk_fputs;
   
   /* start-sanitize-ide */
 #ifdef IDE