This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
[RFA/RFC] printf (stderr,... ->fprintf_unfiltered (gdb_stderr,... in remote*.c
- From: Pierre Muller <muller at cerbere dot u-strasbg dot fr>
- To: gdb-patches at sources dot redhat dot com
- Date: Wed, 24 Apr 2002 15:30:04 +0200
- Subject: [RFA/RFC] printf (stderr,... ->fprintf_unfiltered (gdb_stderr,... in remote*.c
This is one of the files that I listed in
http://sources.redhat.com/ml/gdb/2002-02/msg00212.html
as still having direct uses of stderr.
I don't even know if this is correct,
because I don't know if the gdb_std???
should be used in these files.
remote-array.c seems to also use gdb_stdout
so that one should be correct.
remote-es.c has no other reference to gdb_std???
variables, so here it could well be an error to do that change.
Idem for remote-os9k.c and remote-st.c.
ChangeLog entry:
2002-04-24 Pierre Muller <ics.u-strasbg.fr>
* remote-array.c (printf_monitor, write_monitor,
array_insert_breakpoint, array_remove_breakpoint ):
Replace fprintf (stderr,...
with fprintf_unfiltered (gdb_stderr,....
* remote-es.c: Likewise.
* remote-os9k.c: Likewise.
* remote-st.c: Likewise.
Index: remote-array.c
===================================================================
RCS file: /cvs/src/src/gdb/remote-array.c,v
retrieving revision 1.18
diff -u -p -r1.18 remote-array.c
--- remote-array.c 19 Jan 2002 03:32:37 -0000 1.18
+++ remote-array.c 24 Apr 2002 12:41:19 -0000
@@ -193,7 +193,7 @@ printf_monitor (char *pattern,...)
if (strlen (buf) > PBUFSIZ)
error ("printf_monitor(): string too long");
if (serial_write (array_desc, buf, strlen (buf)))
- fprintf (stderr, "serial_write failed: %s\n", safe_strerror (errno));
+ fprintf_unfiltered (gdb_stderr, "serial_write failed: %s\n", safe_strerror (errno));
}
/*
* write_monitor -- send raw data to monitor.
@@ -202,7 +202,7 @@ static void
write_monitor (char data[], int len)
{
if (serial_write (array_desc, data, len))
- fprintf (stderr, "serial_write failed: %s\n", safe_strerror (errno));
+ fprintf_unfiltered (gdb_stderr, "serial_write failed: %s\n", safe_strerror (errno));
*(data + len + 1) = '\0';
debuglogs (1, "write_monitor(), Sending: \"%s\".", data);
@@ -1053,7 +1053,7 @@ array_insert_breakpoint (CORE_ADDR addr,
}
}
- fprintf (stderr, "Too many breakpoints (> 16) for monitor\n");
+ fprintf_unfiltered (gdb_stderr, "Too many breakpoints (> 16) for monitor\n");
return 1;
}
@@ -1078,7 +1078,7 @@ array_remove_breakpoint (CORE_ADDR addr,
return 0;
}
}
- fprintf (stderr, "Can't find breakpoint associated with 0x%s\n",
+ fprintf_unfiltered (gdb_stderr, "Can't find breakpoint associated with 0x%s\n",
paddr_nz (addr));
return 1;
}
Index: remote-es.c
===================================================================
RCS file: /cvs/src/src/gdb/remote-es.c,v
retrieving revision 1.20
diff -u -p -r1.20 remote-es.c
--- remote-es.c 19 Jan 2002 03:32:37 -0000 1.20
+++ remote-es.c 24 Apr 2002 12:41:19 -0000
@@ -1616,7 +1616,7 @@ send (char *string)
{
if (kiodebug)
{
- fprintf (stderr, "Sending: %s\n", string);
+ fprintf_unfiltered (gdb_stderr, "Sending: %s\n", string);
}
serial_write (es1800_desc, string, strlen (string));
}
@@ -1663,7 +1663,7 @@ getmessage (char *buf, int len)
if (kiodebug)
{
- fprintf (stderr, "message received :%s\n", buf);
+ fprintf_unfiltered (gdb_stderr, "message received :%s\n", buf);
}
}
Index: remote-os9k.c
===================================================================
RCS file: /cvs/src/src/gdb/remote-os9k.c,v
retrieving revision 1.18
diff -u -p -r1.18 remote-os9k.c
--- remote-os9k.c 19 Jan 2002 03:32:38 -0000 1.18
+++ remote-os9k.c 24 Apr 2002 12:41:19 -0000
@@ -101,7 +101,7 @@ printf_monitor (char *pattern,...)
va_end (args);
if (serial_write (monitor_desc, buf, strlen (buf)))
- fprintf (stderr, "serial_write failed: %s\n", safe_strerror (errno));
+ fprintf_unfiltered (gdb_stderr, "serial_write failed: %s\n", safe_strerror (errno));
}
/* Read a character from the remote system, doing all the fancy timeout stuff */
@@ -380,9 +380,9 @@ rombug_close (int quitting)
if (log_file)
{
if (ferror (log_file))
- fprintf (stderr, "Error writing log file.\n");
+ fprintf_unfiltered (gdb_stderr, "Error writing log file.\n");
if (fclose (log_file) != 0)
- fprintf (stderr, "Error closing log file.\n");
+ fprintf_unfiltered (gdb_stderr, "Error closing log file.\n");
log_file = 0;
}
}
@@ -883,7 +883,7 @@ rombug_insert_breakpoint (CORE_ADDR addr
return 0;
}
- fprintf (stderr, "Too many breakpoints (> 16) for monitor\n");
+ fprintf_unfiltered (gdb_stderr, "Too many breakpoints (> 16) for monitor\n");
return 1;
}
@@ -908,7 +908,7 @@ rombug_remove_breakpoint (CORE_ADDR addr
return 0;
}
- fprintf (stderr, "Can't find breakpoint associated with 0x%x\n", addr);
+ fprintf_unfiltered (gdb_stderr, "Can't find breakpoint associated with 0x%x\n", addr);
return 1;
}
@@ -949,7 +949,7 @@ rombug_load (char *arg)
if (serial_write (monitor_desc, buf, bytes_read))
{
- fprintf (stderr, "serial_write failed: (while downloading) %s\n", safe_strerror (errno));
+ fprintf_unfiltered (gdb_stderr, "serial_write failed: (while downloading) %s\n", safe_strerror (errno));
break;
}
i = 0;
Index: remote-st.c
===================================================================
RCS file: /cvs/src/src/gdb/remote-st.c,v
retrieving revision 1.16
diff -u -p -r1.16 remote-st.c
--- remote-st.c 19 Jan 2002 03:32:39 -0000 1.16
+++ remote-st.c 24 Apr 2002 12:41:19 -0000
@@ -79,7 +79,7 @@ printf_stdebug (char *pattern,...)
va_end (args);
if (serial_write (st2000_desc, buf, strlen (buf)))
- fprintf (stderr, "serial_write failed: %s\n", safe_strerror (errno));
+ fprintf_unfiltered (gdb_stderr, "serial_write failed: %s\n", safe_strerror (errno));
}
/* Read a character from the remote system, doing all the fancy timeout
@@ -324,9 +324,9 @@ st2000_close (int quitting)
if (log_file)
{
if (ferror (log_file))
- fprintf (stderr, "Error writing log file.\n");
+ fprintf_unfiltered (gdb_stderr, "Error writing log file.\n");
if (fclose (log_file) != 0)
- fprintf (stderr, "Error closing log file.\n");
+ fprintf_unfiltered (gdb_stderr, "Error closing log file.\n");
}
#endif
}
@@ -616,7 +616,7 @@ st2000_insert_breakpoint (CORE_ADDR addr
return 0;
}
- fprintf (stderr, "Too many breakpoints (> 16) for STDBUG\n");
+ fprintf_unfiltered (gdb_stderr, "Too many breakpoints (> 16) for STDBUG\n");
return 1;
}
@@ -635,7 +635,7 @@ st2000_remove_breakpoint (CORE_ADDR addr
return 0;
}
- fprintf (stderr, "Can't find breakpoint associated with 0x%x\n", addr);
+ fprintf_unfiltered (gdb_stderr, "Can't find breakpoint associated with 0x%x\n", addr);
return 1;
}