* [RFA] remove fprintf (stderr, ...) from dwarf2read.c
@ 2002-02-14 4:04 Pierre Muller
2002-02-14 6:41 ` Daniel Jacobowitz
2002-02-15 13:04 ` Jim Blandy
0 siblings, 2 replies; 4+ messages in thread
From: Pierre Muller @ 2002-02-14 4:04 UTC (permalink / raw)
To: gdb
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 ?
2002-02-14 Pierre Muller <muller@ics.u-strasbg.fr>
dwarf2read.c: Replace fprintf (stderr, ...) by fprintf_unfiltered (gdb_stderr, ...)
Index: dwarf2read.c
===================================================================
RCS file: /cvs/src/src/gdb/dwarf2read.c,v
retrieving revision 1.47
diff -u -p -r1.47 dwarf2read.c
--- dwarf2read.c 2002/01/20 19:42:04 1.47
+++ dwarf2read.c 2002/02/14 11:39:21
@@ -5617,29 +5617,29 @@ dump_die (struct die_info *die)
{
unsigned int i;
- fprintf (stderr, "Die: %s (abbrev = %d, offset = %d)\n",
+ fprintf_unfiltered (gdb_stderr, "Die: %s (abbrev = %d, offset = %d)\n",
dwarf_tag_name (die->tag), die->abbrev, die->offset);
- fprintf (stderr, "\thas children: %s\n",
+ fprintf_unfiltered (gdb_stderr, "\thas children: %s\n",
dwarf_bool_name (die->has_children));
- fprintf (stderr, "\tattributes:\n");
+ fprintf_unfiltered (gdb_stderr, "\tattributes:\n");
for (i = 0; i < die->num_attrs; ++i)
{
- fprintf (stderr, "\t\t%s (%s) ",
+ fprintf_unfiltered (gdb_stderr, "\t\t%s (%s) ",
dwarf_attr_name (die->attrs[i].name),
dwarf_form_name (die->attrs[i].form));
switch (die->attrs[i].form)
{
case DW_FORM_ref_addr:
case DW_FORM_addr:
- fprintf (stderr, "address: ");
+ fprintf_unfiltered (gdb_stderr, "address: ");
print_address_numeric (DW_ADDR (&die->attrs[i]), 1, gdb_stderr);
break;
case DW_FORM_block2:
case DW_FORM_block4:
case DW_FORM_block:
case DW_FORM_block1:
- fprintf (stderr, "block: size %d", DW_BLOCK (&die->attrs[i])->size);
+ fprintf_unfiltered (gdb_stderr, "block: size %d", DW_BLOCK (&die->attr
s[i])->size);
break;
case DW_FORM_data1:
case DW_FORM_data2:
@@ -5650,30 +5650,30 @@ dump_die (struct die_info *die)
case DW_FORM_ref4:
case DW_FORM_udata:
case DW_FORM_sdata:
- fprintf (stderr, "constant: %ld", DW_UNSND (&die->attrs[i]));
+ fprintf_unfiltered (gdb_stderr, "constant: %ld", DW_UNSND (&die->attrs
[i]));
break;
case DW_FORM_string:
case DW_FORM_strp:
- fprintf (stderr, "string: \"%s\"",
+ fprintf_unfiltered (gdb_stderr, "string: \"%s\"",
DW_STRING (&die->attrs[i])
? DW_STRING (&die->attrs[i]) : "");
break;
case DW_FORM_flag:
if (DW_UNSND (&die->attrs[i]))
- fprintf (stderr, "flag: TRUE");
+ fprintf_unfiltered (gdb_stderr, "flag: TRUE");
else
- fprintf (stderr, "flag: FALSE");
+ fprintf_unfiltered (gdb_stderr, "flag: FALSE");
break;
case DW_FORM_indirect:
/* the reader will have reduced the indirect form to
the "base form" so this form should not occur */
- fprintf (stderr, "unexpected attribute form: DW_FORM_indirect");
+ fprintf_unfiltered (gdb_stderr, "unexpected attribute form: DW_FORM_in
direct");
break;
default:
- fprintf (stderr, "unsupported attribute form: %d.",
+ fprintf_unfiltered (gdb_stderr, "unsupported attribute form: %d.",
die->attrs[i].form);
}
- fprintf (stderr, "\n");
+ fprintf_unfiltered (gdb_stderr, "\n");
}
}
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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RFA] remove fprintf (stderr, ...) from dwarf2read.c
2002-02-14 4:04 [RFA] remove fprintf (stderr, ...) from dwarf2read.c Pierre Muller
@ 2002-02-14 6:41 ` Daniel Jacobowitz
2002-02-15 13:04 ` Jim Blandy
1 sibling, 0 replies; 4+ messages in thread
From: Daniel Jacobowitz @ 2002-02-14 6:41 UTC (permalink / raw)
To: Pierre Muller; +Cc: gdb
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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RFA] remove fprintf (stderr, ...) from dwarf2read.c
2002-02-14 4:04 [RFA] remove fprintf (stderr, ...) from dwarf2read.c Pierre Muller
2002-02-14 6:41 ` Daniel Jacobowitz
@ 2002-02-15 13:04 ` Jim Blandy
2002-02-15 23:20 ` muller
1 sibling, 1 reply; 4+ messages in thread
From: Jim Blandy @ 2002-02-15 13:04 UTC (permalink / raw)
To: Pierre Muller; +Cc: gdb
Pierre Muller <muller@cerbere.u-strasbg.fr> writes:
> 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.
It's great to be cautious, but you should still ask for this sort of
permission on gdb-patches, not gdb.
Please feel free to commit this change.
>
> 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 ?
>
>
> 2002-02-14 Pierre Muller <muller@ics.u-strasbg.fr>
> dwarf2read.c: Replace fprintf (stderr, ...) by fprintf_unfiltered (gdb_stderr, ...)
>
>
> Index: dwarf2read.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/dwarf2read.c,v
> retrieving revision 1.47
> diff -u -p -r1.47 dwarf2read.c
> --- dwarf2read.c 2002/01/20 19:42:04 1.47
> +++ dwarf2read.c 2002/02/14 11:39:21
> @@ -5617,29 +5617,29 @@ dump_die (struct die_info *die)
> {
> unsigned int i;
>
> - fprintf (stderr, "Die: %s (abbrev = %d, offset = %d)\n",
> + fprintf_unfiltered (gdb_stderr, "Die: %s (abbrev = %d, offset = %d)\n",
> dwarf_tag_name (die->tag), die->abbrev, die->offset);
> - fprintf (stderr, "\thas children: %s\n",
> + fprintf_unfiltered (gdb_stderr, "\thas children: %s\n",
> dwarf_bool_name (die->has_children));
>
> - fprintf (stderr, "\tattributes:\n");
> + fprintf_unfiltered (gdb_stderr, "\tattributes:\n");
> for (i = 0; i < die->num_attrs; ++i)
> {
> - fprintf (stderr, "\t\t%s (%s) ",
> + fprintf_unfiltered (gdb_stderr, "\t\t%s (%s) ",
> dwarf_attr_name (die->attrs[i].name),
> dwarf_form_name (die->attrs[i].form));
> switch (die->attrs[i].form)
> {
> case DW_FORM_ref_addr:
> case DW_FORM_addr:
> - fprintf (stderr, "address: ");
> + fprintf_unfiltered (gdb_stderr, "address: ");
> print_address_numeric (DW_ADDR (&die->attrs[i]), 1, gdb_stderr);
> break;
> case DW_FORM_block2:
> case DW_FORM_block4:
> case DW_FORM_block:
> case DW_FORM_block1:
> - fprintf (stderr, "block: size %d", DW_BLOCK (&die->attrs[i])->size);
> + fprintf_unfiltered (gdb_stderr, "block: size %d", DW_BLOCK (&die->attr
> s[i])->size);
> break;
> case DW_FORM_data1:
> case DW_FORM_data2:
> @@ -5650,30 +5650,30 @@ dump_die (struct die_info *die)
> case DW_FORM_ref4:
> case DW_FORM_udata:
> case DW_FORM_sdata:
> - fprintf (stderr, "constant: %ld", DW_UNSND (&die->attrs[i]));
> + fprintf_unfiltered (gdb_stderr, "constant: %ld", DW_UNSND (&die->attrs
> [i]));
> break;
> case DW_FORM_string:
> case DW_FORM_strp:
> - fprintf (stderr, "string: \"%s\"",
> + fprintf_unfiltered (gdb_stderr, "string: \"%s\"",
> DW_STRING (&die->attrs[i])
> ? DW_STRING (&die->attrs[i]) : "");
> break;
> case DW_FORM_flag:
> if (DW_UNSND (&die->attrs[i]))
> - fprintf (stderr, "flag: TRUE");
> + fprintf_unfiltered (gdb_stderr, "flag: TRUE");
> else
> - fprintf (stderr, "flag: FALSE");
> + fprintf_unfiltered (gdb_stderr, "flag: FALSE");
> break;
> case DW_FORM_indirect:
> /* the reader will have reduced the indirect form to
> the "base form" so this form should not occur */
> - fprintf (stderr, "unexpected attribute form: DW_FORM_indirect");
> + fprintf_unfiltered (gdb_stderr, "unexpected attribute form: DW_FORM_in
> direct");
> break;
> default:
> - fprintf (stderr, "unsupported attribute form: %d.",
> + fprintf_unfiltered (gdb_stderr, "unsupported attribute form: %d.",
> die->attrs[i].form);
> }
> - fprintf (stderr, "\n");
> + fprintf_unfiltered (gdb_stderr, "\n");
> }
> }
>
>
> 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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RFA] remove fprintf (stderr, ...) from dwarf2read.c
2002-02-15 13:04 ` Jim Blandy
@ 2002-02-15 23:20 ` muller
0 siblings, 0 replies; 4+ messages in thread
From: muller @ 2002-02-15 23:20 UTC (permalink / raw)
To: Jim Blandy; +Cc: gdb
At 16:06 15/02/02 -0500, Jim Blandy wrote:
>
>Pierre Muller <muller@cerbere.u-strasbg.fr> writes:
>> 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.
>
>It's great to be cautious, but you should still ask for this sort of
>permission on gdb-patches, not gdb.
Whoops, this was unintended.
>Please feel free to commit this change.
Done.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2002-02-16 7:20 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-02-14 4:04 [RFA] remove fprintf (stderr, ...) from dwarf2read.c Pierre Muller
2002-02-14 6:41 ` Daniel Jacobowitz
2002-02-15 13:04 ` Jim Blandy
2002-02-15 23:20 ` muller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox