This is the mail archive of the
gdb@sources.redhat.com
mailing list for the GDB project.
Re: [RFA] remove fprintf (stderr, ...) from dwarf2read.c
- From: Daniel Jacobowitz <drow at mvista dot com>
- To: Pierre Muller <muller at cerbere dot u-strasbg dot fr>
- Cc: gdb at sources dot redhat dot com
- Date: Thu, 14 Feb 2002 09:41:46 -0500
- Subject: Re: [RFA] remove fprintf (stderr, ...) from dwarf2read.c
- References: <4.2.0.58.20020214123916.019a6840@ics.u-strasbg.fr>
On Thu, Feb 14, 2002 at 01:04:18PM +0100, Pierre Muller wrote:
> Probably the obvious rule could apply here,
> but since my last "obvious fix" was a total fiasco,
> I prefer to ask once again for permission here.
>
> There are several other files that use
> fprintf (stderr,...)
>
> Here is the complete list I found:
> arm-tdep.c
> f-lang.c
> hpread.c
> i386-stub.c
> m68k-stub.c
> remote-array.c
> remote-es.c
> remote-os9k.c
> remote-st.c
> stop-gdb.c
> symm-nat.c
> gdbserver/gdbreplay.c
> gdbserver/linux-low.c
> gdbserver/low-hppabsd.c
> gdbserver/low-lynx.c
> gdbserver/low-nbsd.c
> gdbserver/low-sim.c
> gdbserver/low-sparc.c
> gdbserver/low-sun3.c
> gdbserver/remote-utils.c
> gdbserver/server.c
> gdbserver/utils.c
> nindy-share/Onindy.c
> nindy-share/nindy.c
> nlm/gdbserve.c
> rdi-share/devsw.c
> rdi-share/etherdrv.c
> rdi-share/hsys.c
>
>
> Some of these files, like i386-stub.c and m68k-stub.c don't even use defs.h
> (which contains the declaration of fprintf_unfiltered)
> and thus should not be changed but for most others (at least at gdb dir level)
> should be changed, no ?
Some of those belong to GDB and some do not. gdbserver/* do not;
i386-stub.c and m68k-stub.c do not, I believe. Most of the others do,
but it's worth verifying - nlm/gdbserve.c looks suspicious, for
example.
--
Daniel Jacobowitz Carnegie Mellon University
MontaVista Software Debian GNU/Linux Developer