This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
[RFC/RFA] avoid spurious Watchpoint X output on cygwin native target.
- From: Pierre Muller <muller at ics dot u-strasbg dot fr>
- To: gdb-patches at sources dot redhat dot com
- Date: Mon, 15 Jul 2002 18:15:01 +0200
- Subject: [RFC/RFA] avoid spurious Watchpoint X output on cygwin native target.
The following patch suppresses all
spurious output when a DLL is loaded in native cygwin GDB.
Its a RFC for two reasons:
First reason:
Question: Why do we currently get ouptut while loading a DLL
while the safe_symbol_file_add does redefine gdb_stdout and
gdbstderr to dummy output exactly to suppress all output?
Answer: Because, despite gdb_stdout is redefined as a
dummy file does noting with the strings it gets
(and thus is a correct way of suppressing ouptut
sent to gdb_stdout), the Watchpoint X...
message is sent to uiout struct
(see mention function in breakpoint.c source)
The only question here is if a change of gdb_stdout should not
also change the global uiout variable behavior.
This could easily be achieved by replacing
the stream field of the data field of ui_out struct into a
'** ui_file' instead of a simple '*ui_file'.
Is that complete nonsense, or does it seem logical to someone?
(One argument for this is that you get the same unwanted output on
loading of shared libraries on linux for instance ...)
Here is the win32 specific patch and the associated changelog.
2nd reason: It does not free the uiout struct created because I didn't find
how to free this cleanly.
200-07-15 Pierre Muller <muller@ics.u-strasbg.fr>
* win32-nat.c: avoid unwanted ouptput about watchpoints on DLL loadings.
(struct safe_symbol_file_add_args): New field uiout to store
previous value of global uiout variable.
(safe_symbol_file_add_cleanup): restore uiout value.
(safe_symbol_file_add): store uiout and create a dummy one.
$ cvs diff -u -p win32-nat.c
Index: win32-nat.c
===================================================================
RCS file: /cvs/src/src/gdb/win32-nat.c,v
retrieving revision 1.60
diff -u -p -r1.60 win32-nat.c
--- win32-nat.c 11 Jul 2002 13:50:49 -0000 1.60
+++ win32-nat.c 15 Jul 2002 16:10:42 -0000
@@ -45,6 +45,9 @@
#include <imagehlp.h>
#include <sys/cygwin.h>
+
+#include "ui-out.h"
+#include "cli-out.h"
#include "buildsym.h"
#include "symfile.h"
#include "objfiles.h"
@@ -494,6 +497,7 @@ struct safe_symbol_file_add_args
int mainline;
int flags;
struct ui_file *err, *out;
+ struct ui_out *uiout;
struct objfile *ret;
};
@@ -534,6 +538,7 @@ safe_symbol_file_add_cleanup (void *p)
ui_file_delete (gdb_stdout);
gdb_stderr = sp->err;
gdb_stdout = sp->out;
+ uiout = sp->uiout;
#undef sp
}
@@ -550,10 +555,12 @@ safe_symbol_file_add (char *name, int fr
p.err = gdb_stderr;
p.out = gdb_stdout;
+ p.uiout = uiout;
gdb_flush (gdb_stderr);
gdb_flush (gdb_stdout);
gdb_stderr = ui_file_new ();
gdb_stdout = ui_file_new ();
+ uiout = cli_out_new (gdb_stdout);
p.name = name;
p.from_tty = from_tty;
p.addrs = addrs;
Pierre Muller
Institut Charles Sadron
6,rue Boussingault
F 67083 STRASBOURG CEDEX (France)
mailto:muller@ics.u-strasbg.fr
Phone : (33)-3-88-41-40-07 Fax : (33)-3-88-41-40-99