Use correct stream for process record output
authorTom Tromey <tom@tromey.com>
Sun, 26 Dec 2021 20:41:36 +0000 (13:41 -0700)
committerTom Tromey <tom@tromey.com>
Wed, 29 Dec 2021 17:40:10 +0000 (10:40 -0700)
commitd68510ac191cb8b4b662a5fd1f3bcf9f4b136971
treeab8d361f18255facf28e086d9a102515e123fa13
parent0fed74615b0f263c359a13a7a17ade36a986ddd1
Use correct stream for process record output

The process record code often emits unfiltered output.  In some cases,
this output ought to go to gdb_stderr (but see below).  In other
cases, the output is guarded by a logging variable and so ought to go
to gdb_stdlog.  This patch makes these changes.

Note that in many cases, the output to stderr is followed by a
"return -1", which is how process record indicates an error.  It seems
to me that calling error here would be preferable, because, in many
cases, that's all the caller does when it sees a -1.  However, I
haven't made this change.

This is part of PR gdb/7233.

Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=7233
13 files changed:
gdb/aarch64-linux-tdep.c
gdb/aarch64-tdep.c
gdb/amd64-linux-tdep.c
gdb/arm-linux-tdep.c
gdb/arm-tdep.c
gdb/i386-linux-tdep.c
gdb/i386-tdep.c
gdb/linux-record.c
gdb/moxie-tdep.c
gdb/ppc-linux-tdep.c
gdb/rs6000-tdep.c
gdb/s390-linux-tdep.c
gdb/s390-tdep.c