[4/6] gdb: make debug_target use one-liners

Message ID 20240417205426.2030615-4-simon.marchi@polymtl.ca
State New
Headers
Series [1/6] gdb: add includes in target-debug.h |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_gdb_build--master-aarch64 success Testing passed
linaro-tcwg-bot/tcwg_gdb_build--master-arm success Testing passed
linaro-tcwg-bot/tcwg_gdb_check--master-arm success Testing passed
linaro-tcwg-bot/tcwg_gdb_check--master-aarch64 success Testing passed

Commit Message

Simon Marchi April 17, 2024, 8:54 p.m. UTC
  From: Simon Marchi <simon.marchi@efficios.com>

Turn the debug prints in debug_target's method to be one liners.  For
instance, change this:

    gdb_printf (gdb_stdlog, "<- %s->wait (", this->beneath ()->shortname ());
    gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
    gdb_puts (", ", gdb_stdlog);
    gdb_puts (target_debug_print_target_waitstatus_p (arg1), gdb_stdlog);
    gdb_puts (", ", gdb_stdlog);
    gdb_puts (target_debug_print_target_wait_flags (arg2), gdb_stdlog);
    gdb_puts (") = ", gdb_stdlog);
    target_debug_print_ptid_t (result);
    gdb_puts ("\n", gdb_stdlog);

into this:

    gdb_printf (gdb_stdlog,
               "<- %s->wait (%s, %s, %s) = %s\n",
               this->beneath ()->shortname (),
               target_debug_print_ptid_t (arg0).c_str (),
               target_debug_print_target_waitstatus_p (arg1).c_str (),
               target_debug_print_target_wait_flags (arg2).c_str (),
               target_debug_print_ptid_t (result).c_str ());

This makes it possible for a subsequent patch to turn this gdb_printf
call into a `target_debug_printf` call.

Change-Id: I808202438972fac1bba2f8ccb63e66a4fcef20c9
---
 gdb/make-target-delegates.py |   44 +-
 gdb/target-debug.h           |    4 +-
 gdb/target-delegates.c       | 1651 +++++++++++++++++-----------------
 3 files changed, 835 insertions(+), 864 deletions(-)
  

Comments

Tom Tromey April 19, 2024, 7:23 p.m. UTC | #1
>>>>> "Simon" == Simon Marchi <simon.marchi@polymtl.ca> writes:

Simon> From: Simon Marchi <simon.marchi@efficios.com>
Simon> Turn the debug prints in debug_target's method to be one liners.

Ok.
Approved-By: Tom Tromey <tom@tromey.com>

Tom
  

Patch

diff --git a/gdb/make-target-delegates.py b/gdb/make-target-delegates.py
index 051efce94dab..f6f2bf004ff5 100755
--- a/gdb/make-target-delegates.py
+++ b/gdb/make-target-delegates.py
@@ -290,28 +290,34 @@  def write_debugmethod(
     print(", ".join(names), file=f, end="")
     print(");", file=f)
 
-    # Now print the arguments.
-    print(
-        '  gdb_printf (gdb_stdlog, "<- %s->'
-        + name
-        + ' (", this->beneath ()->shortname ());',
-        file=f,
+    # Generate the debug printf call.
+    args_fmt = ", ".join(["%s"] * len(argtypes))
+    args = "".join(
+        [
+            (",\n\t      {printer} (arg{i}).c_str ()").format(
+                printer=munge_type(t), i=i
+            )
+            for i, t in enumerate(argtypes)
+        ]
     )
-    for i in range(len(argtypes)):
-        if i > 0:
-            print('  gdb_puts (", ", gdb_stdlog);', file=f)
-        printer = munge_type(argtypes[i])
-        print(
-            "  gdb_puts (" + printer + " (" + names[i] + "), gdb_stdlog);",
-            file=f,
-        )
+
     if return_type != "void":
-        print('  gdb_puts (") = ", gdb_stdlog);', file=f)
-        printer = munge_type(return_type)
-        print("  " + printer + " (result);", file=f)
-        print('  gdb_puts ("\\n", gdb_stdlog);', file=f)
+        ret_fmt = " = %s"
+        ret = ",\n\t      {printer} (result).c_str ()".format(
+            printer=munge_type(return_type)
+        )
     else:
-        print('  gdb_puts (")\\n", gdb_stdlog);', file=f)
+        ret_fmt = ""
+        ret = ""
+
+    print(
+        (
+            "  gdb_printf (gdb_stdlog,\n"
+            '\t      "<- %s->{name} ({args_fmt}){ret_fmt}\\n",\n'
+            "\t      this->beneath ()->shortname (){args}{ret});"
+        ).format(name=name, args_fmt=args_fmt, args=args, ret_fmt=ret_fmt, ret=ret),
+        file=f,
+    )
 
     if return_type != "void":
         print("  return result;", file=f)
diff --git a/gdb/target-debug.h b/gdb/target-debug.h
index 1ed4517f7ffc..c52a71aa439e 100644
--- a/gdb/target-debug.h
+++ b/gdb/target-debug.h
@@ -308,7 +308,7 @@  target_debug_print_target_waitstatus_p (struct target_waitstatus *status)
 
 /* Functions that are used via TARGET_DEBUG_PRINTER.  */
 
-static const char *
+static std::string
 target_debug_print_step (int step)
 { return step ? "step" : "continue"; }
 
@@ -331,7 +331,7 @@  target_debug_print_signals (gdb::array_view<const unsigned char> sigs)
   return s;
 }
 
-static const char *
+static std::string
 target_debug_print_size_t (size_t size)
 {
   return pulongest (size);
diff --git a/gdb/target-delegates.c b/gdb/target-delegates.c
index f6e71f32e67b..f4f88a952157 100644
--- a/gdb/target-delegates.c
+++ b/gdb/target-delegates.c
@@ -392,9 +392,10 @@  debug_target::post_attach (int arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->post_attach (...)\n", this->beneath ()->shortname ());
   this->beneath ()->post_attach (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->post_attach (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->post_attach (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str ());
 }
 
 void
@@ -413,11 +414,11 @@  debug_target::detach (inferior *arg0, int arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->detach (...)\n", this->beneath ()->shortname ());
   this->beneath ()->detach (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->detach (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_inferior_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->detach (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_inferior_p (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str ());
 }
 
 void
@@ -437,11 +438,11 @@  debug_target::disconnect (const char *arg0, int arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->disconnect (...)\n", this->beneath ()->shortname ());
   this->beneath ()->disconnect (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->disconnect (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_char_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->disconnect (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_char_p (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str ());
 }
 
 void
@@ -461,13 +462,12 @@  debug_target::resume (ptid_t arg0, int arg1, enum gdb_signal arg2)
 {
   gdb_printf (gdb_stdlog, "-> %s->resume (...)\n", this->beneath ()->shortname ());
   this->beneath ()->resume (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->resume (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_step (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_gdb_signal (arg2), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->resume (%s, %s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str (),
+	      target_debug_print_step (arg1).c_str (),
+	      target_debug_print_gdb_signal (arg2).c_str ());
 }
 
 void
@@ -486,8 +486,9 @@  debug_target::commit_resumed ()
 {
   gdb_printf (gdb_stdlog, "-> %s->commit_resumed (...)\n", this->beneath ()->shortname ());
   this->beneath ()->commit_resumed ();
-  gdb_printf (gdb_stdlog, "<- %s->commit_resumed (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->commit_resumed ()\n",
+	      this->beneath ()->shortname ());
 }
 
 ptid_t
@@ -508,15 +509,13 @@  debug_target::wait (ptid_t arg0, struct target_waitstatus *arg1, target_wait_fla
   gdb_printf (gdb_stdlog, "-> %s->wait (...)\n", this->beneath ()->shortname ());
   ptid_t result
     = this->beneath ()->wait (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->wait (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_target_waitstatus_p (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_target_wait_flags (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_ptid_t (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->wait (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str (),
+	      target_debug_print_target_waitstatus_p (arg1).c_str (),
+	      target_debug_print_target_wait_flags (arg2).c_str (),
+	      target_debug_print_ptid_t (result).c_str ());
   return result;
 }
 
@@ -536,11 +535,11 @@  debug_target::fetch_registers (struct regcache *arg0, int arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->fetch_registers (...)\n", this->beneath ()->shortname ());
   this->beneath ()->fetch_registers (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->fetch_registers (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_regcache_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->fetch_registers (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_regcache_p (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str ());
 }
 
 void
@@ -560,11 +559,11 @@  debug_target::store_registers (struct regcache *arg0, int arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->store_registers (...)\n", this->beneath ()->shortname ());
   this->beneath ()->store_registers (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->store_registers (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_regcache_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->store_registers (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_regcache_p (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str ());
 }
 
 void
@@ -584,9 +583,10 @@  debug_target::prepare_to_store (struct regcache *arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->prepare_to_store (...)\n", this->beneath ()->shortname ());
   this->beneath ()->prepare_to_store (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->prepare_to_store (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_regcache_p (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->prepare_to_store (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_regcache_p (arg0).c_str ());
 }
 
 void
@@ -605,8 +605,9 @@  debug_target::files_info ()
 {
   gdb_printf (gdb_stdlog, "-> %s->files_info (...)\n", this->beneath ()->shortname ());
   this->beneath ()->files_info ();
-  gdb_printf (gdb_stdlog, "<- %s->files_info (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->files_info ()\n",
+	      this->beneath ()->shortname ());
 }
 
 int
@@ -627,13 +628,12 @@  debug_target::insert_breakpoint (struct gdbarch *arg0, struct bp_target_info *ar
   gdb_printf (gdb_stdlog, "-> %s->insert_breakpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->insert_breakpoint (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->insert_breakpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_gdbarch_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_bp_target_info_p (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->insert_breakpoint (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_gdbarch_p (arg0).c_str (),
+	      target_debug_print_bp_target_info_p (arg1).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -655,15 +655,13 @@  debug_target::remove_breakpoint (struct gdbarch *arg0, struct bp_target_info *ar
   gdb_printf (gdb_stdlog, "-> %s->remove_breakpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->remove_breakpoint (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->remove_breakpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_gdbarch_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_bp_target_info_p (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_remove_bp_reason (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->remove_breakpoint (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_gdbarch_p (arg0).c_str (),
+	      target_debug_print_bp_target_info_p (arg1).c_str (),
+	      target_debug_print_remove_bp_reason (arg2).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -685,10 +683,10 @@  debug_target::stopped_by_sw_breakpoint ()
   gdb_printf (gdb_stdlog, "-> %s->stopped_by_sw_breakpoint (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->stopped_by_sw_breakpoint ();
-  gdb_printf (gdb_stdlog, "<- %s->stopped_by_sw_breakpoint (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->stopped_by_sw_breakpoint () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -710,10 +708,10 @@  debug_target::supports_stopped_by_sw_breakpoint ()
   gdb_printf (gdb_stdlog, "-> %s->supports_stopped_by_sw_breakpoint (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_stopped_by_sw_breakpoint ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_stopped_by_sw_breakpoint (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_stopped_by_sw_breakpoint () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -735,10 +733,10 @@  debug_target::stopped_by_hw_breakpoint ()
   gdb_printf (gdb_stdlog, "-> %s->stopped_by_hw_breakpoint (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->stopped_by_hw_breakpoint ();
-  gdb_printf (gdb_stdlog, "<- %s->stopped_by_hw_breakpoint (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->stopped_by_hw_breakpoint () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -760,10 +758,10 @@  debug_target::supports_stopped_by_hw_breakpoint ()
   gdb_printf (gdb_stdlog, "-> %s->supports_stopped_by_hw_breakpoint (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_stopped_by_hw_breakpoint ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_stopped_by_hw_breakpoint (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_stopped_by_hw_breakpoint () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -785,15 +783,13 @@  debug_target::can_use_hw_breakpoint (enum bptype arg0, int arg1, int arg2)
   gdb_printf (gdb_stdlog, "-> %s->can_use_hw_breakpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->can_use_hw_breakpoint (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->can_use_hw_breakpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_bptype (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->can_use_hw_breakpoint (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bptype (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str (),
+	      target_debug_print_int (arg2).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -815,10 +811,10 @@  debug_target::ranged_break_num_registers ()
   gdb_printf (gdb_stdlog, "-> %s->ranged_break_num_registers (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->ranged_break_num_registers ();
-  gdb_printf (gdb_stdlog, "<- %s->ranged_break_num_registers (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->ranged_break_num_registers () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -840,13 +836,12 @@  debug_target::insert_hw_breakpoint (struct gdbarch *arg0, struct bp_target_info
   gdb_printf (gdb_stdlog, "-> %s->insert_hw_breakpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->insert_hw_breakpoint (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->insert_hw_breakpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_gdbarch_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_bp_target_info_p (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->insert_hw_breakpoint (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_gdbarch_p (arg0).c_str (),
+	      target_debug_print_bp_target_info_p (arg1).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -868,13 +863,12 @@  debug_target::remove_hw_breakpoint (struct gdbarch *arg0, struct bp_target_info
   gdb_printf (gdb_stdlog, "-> %s->remove_hw_breakpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->remove_hw_breakpoint (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->remove_hw_breakpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_gdbarch_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_bp_target_info_p (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->remove_hw_breakpoint (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_gdbarch_p (arg0).c_str (),
+	      target_debug_print_bp_target_info_p (arg1).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -896,17 +890,14 @@  debug_target::remove_watchpoint (CORE_ADDR arg0, int arg1, enum target_hw_bp_typ
   gdb_printf (gdb_stdlog, "-> %s->remove_watchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->remove_watchpoint (arg0, arg1, arg2, arg3);
-  gdb_printf (gdb_stdlog, "<- %s->remove_watchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_target_hw_bp_type (arg2), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_expression_p (arg3), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->remove_watchpoint (%s, %s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str (),
+	      target_debug_print_target_hw_bp_type (arg2).c_str (),
+	      target_debug_print_expression_p (arg3).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -928,17 +919,14 @@  debug_target::insert_watchpoint (CORE_ADDR arg0, int arg1, enum target_hw_bp_typ
   gdb_printf (gdb_stdlog, "-> %s->insert_watchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->insert_watchpoint (arg0, arg1, arg2, arg3);
-  gdb_printf (gdb_stdlog, "<- %s->insert_watchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_target_hw_bp_type (arg2), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_expression_p (arg3), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->insert_watchpoint (%s, %s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str (),
+	      target_debug_print_target_hw_bp_type (arg2).c_str (),
+	      target_debug_print_expression_p (arg3).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -960,15 +948,13 @@  debug_target::insert_mask_watchpoint (CORE_ADDR arg0, CORE_ADDR arg1, enum targe
   gdb_printf (gdb_stdlog, "-> %s->insert_mask_watchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->insert_mask_watchpoint (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->insert_mask_watchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_target_hw_bp_type (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->insert_mask_watchpoint (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_CORE_ADDR (arg1).c_str (),
+	      target_debug_print_target_hw_bp_type (arg2).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -990,15 +976,13 @@  debug_target::remove_mask_watchpoint (CORE_ADDR arg0, CORE_ADDR arg1, enum targe
   gdb_printf (gdb_stdlog, "-> %s->remove_mask_watchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->remove_mask_watchpoint (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->remove_mask_watchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_target_hw_bp_type (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->remove_mask_watchpoint (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_CORE_ADDR (arg1).c_str (),
+	      target_debug_print_target_hw_bp_type (arg2).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -1020,10 +1004,10 @@  debug_target::stopped_by_watchpoint ()
   gdb_printf (gdb_stdlog, "-> %s->stopped_by_watchpoint (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->stopped_by_watchpoint ();
-  gdb_printf (gdb_stdlog, "<- %s->stopped_by_watchpoint (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->stopped_by_watchpoint () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -1045,10 +1029,10 @@  debug_target::have_steppable_watchpoint ()
   gdb_printf (gdb_stdlog, "-> %s->have_steppable_watchpoint (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->have_steppable_watchpoint ();
-  gdb_printf (gdb_stdlog, "<- %s->have_steppable_watchpoint (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->have_steppable_watchpoint () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -1070,11 +1054,11 @@  debug_target::stopped_data_address (CORE_ADDR *arg0)
   gdb_printf (gdb_stdlog, "-> %s->stopped_data_address (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->stopped_data_address (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->stopped_data_address (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR_p (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->stopped_data_address (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR_p (arg0).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -1096,15 +1080,13 @@  debug_target::watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, int
   gdb_printf (gdb_stdlog, "-> %s->watchpoint_addr_within_range (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->watchpoint_addr_within_range (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->watchpoint_addr_within_range (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->watchpoint_addr_within_range (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_CORE_ADDR (arg1).c_str (),
+	      target_debug_print_int (arg2).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -1126,13 +1108,12 @@  debug_target::region_ok_for_hw_watchpoint (CORE_ADDR arg0, int arg1)
   gdb_printf (gdb_stdlog, "-> %s->region_ok_for_hw_watchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->region_ok_for_hw_watchpoint (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->region_ok_for_hw_watchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->region_ok_for_hw_watchpoint (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -1154,17 +1135,14 @@  debug_target::can_accel_watchpoint_condition (CORE_ADDR arg0, int arg1, int arg2
   gdb_printf (gdb_stdlog, "-> %s->can_accel_watchpoint_condition (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->can_accel_watchpoint_condition (arg0, arg1, arg2, arg3);
-  gdb_printf (gdb_stdlog, "<- %s->can_accel_watchpoint_condition (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg2), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_expression_p (arg3), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->can_accel_watchpoint_condition (%s, %s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str (),
+	      target_debug_print_int (arg2).c_str (),
+	      target_debug_print_expression_p (arg3).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -1186,13 +1164,12 @@  debug_target::masked_watch_num_registers (CORE_ADDR arg0, CORE_ADDR arg1)
   gdb_printf (gdb_stdlog, "-> %s->masked_watch_num_registers (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->masked_watch_num_registers (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->masked_watch_num_registers (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->masked_watch_num_registers (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_CORE_ADDR (arg1).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -1214,10 +1191,10 @@  debug_target::can_do_single_step ()
   gdb_printf (gdb_stdlog, "-> %s->can_do_single_step (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->can_do_single_step ();
-  gdb_printf (gdb_stdlog, "<- %s->can_do_single_step (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->can_do_single_step () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -1239,10 +1216,10 @@  debug_target::supports_terminal_ours ()
   gdb_printf (gdb_stdlog, "-> %s->supports_terminal_ours (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_terminal_ours ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_terminal_ours (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_terminal_ours () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -1262,8 +1239,9 @@  debug_target::terminal_init ()
 {
   gdb_printf (gdb_stdlog, "-> %s->terminal_init (...)\n", this->beneath ()->shortname ());
   this->beneath ()->terminal_init ();
-  gdb_printf (gdb_stdlog, "<- %s->terminal_init (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->terminal_init ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -1282,8 +1260,9 @@  debug_target::terminal_inferior ()
 {
   gdb_printf (gdb_stdlog, "-> %s->terminal_inferior (...)\n", this->beneath ()->shortname ());
   this->beneath ()->terminal_inferior ();
-  gdb_printf (gdb_stdlog, "<- %s->terminal_inferior (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->terminal_inferior ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -1302,8 +1281,9 @@  debug_target::terminal_save_inferior ()
 {
   gdb_printf (gdb_stdlog, "-> %s->terminal_save_inferior (...)\n", this->beneath ()->shortname ());
   this->beneath ()->terminal_save_inferior ();
-  gdb_printf (gdb_stdlog, "<- %s->terminal_save_inferior (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->terminal_save_inferior ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -1322,8 +1302,9 @@  debug_target::terminal_ours_for_output ()
 {
   gdb_printf (gdb_stdlog, "-> %s->terminal_ours_for_output (...)\n", this->beneath ()->shortname ());
   this->beneath ()->terminal_ours_for_output ();
-  gdb_printf (gdb_stdlog, "<- %s->terminal_ours_for_output (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->terminal_ours_for_output ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -1342,8 +1323,9 @@  debug_target::terminal_ours ()
 {
   gdb_printf (gdb_stdlog, "-> %s->terminal_ours (...)\n", this->beneath ()->shortname ());
   this->beneath ()->terminal_ours ();
-  gdb_printf (gdb_stdlog, "<- %s->terminal_ours (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->terminal_ours ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -1363,11 +1345,11 @@  debug_target::terminal_info (const char *arg0, int arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->terminal_info (...)\n", this->beneath ()->shortname ());
   this->beneath ()->terminal_info (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->terminal_info (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_char_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->terminal_info (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_char_p (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str ());
 }
 
 void
@@ -1387,8 +1369,9 @@  debug_target::kill ()
 {
   gdb_printf (gdb_stdlog, "-> %s->kill (...)\n", this->beneath ()->shortname ());
   this->beneath ()->kill ();
-  gdb_printf (gdb_stdlog, "<- %s->kill (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->kill ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -1408,11 +1391,11 @@  debug_target::load (const char *arg0, int arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->load (...)\n", this->beneath ()->shortname ());
   this->beneath ()->load (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->load (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_char_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->load (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_char_p (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str ());
 }
 
 int
@@ -1433,11 +1416,11 @@  debug_target::insert_fork_catchpoint (int arg0)
   gdb_printf (gdb_stdlog, "-> %s->insert_fork_catchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->insert_fork_catchpoint (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->insert_fork_catchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->insert_fork_catchpoint (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -1459,11 +1442,11 @@  debug_target::remove_fork_catchpoint (int arg0)
   gdb_printf (gdb_stdlog, "-> %s->remove_fork_catchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->remove_fork_catchpoint (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->remove_fork_catchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->remove_fork_catchpoint (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -1485,11 +1468,11 @@  debug_target::insert_vfork_catchpoint (int arg0)
   gdb_printf (gdb_stdlog, "-> %s->insert_vfork_catchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->insert_vfork_catchpoint (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->insert_vfork_catchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->insert_vfork_catchpoint (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -1511,11 +1494,11 @@  debug_target::remove_vfork_catchpoint (int arg0)
   gdb_printf (gdb_stdlog, "-> %s->remove_vfork_catchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->remove_vfork_catchpoint (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->remove_vfork_catchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->remove_vfork_catchpoint (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -1536,17 +1519,14 @@  debug_target::follow_fork (inferior *arg0, ptid_t arg1, target_waitkind arg2, bo
 {
   gdb_printf (gdb_stdlog, "-> %s->follow_fork (...)\n", this->beneath ()->shortname ());
   this->beneath ()->follow_fork (arg0, arg1, arg2, arg3, arg4);
-  gdb_printf (gdb_stdlog, "<- %s->follow_fork (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_inferior_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ptid_t (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_target_waitkind (arg2), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_bool (arg3), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_bool (arg4), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->follow_fork (%s, %s, %s, %s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_inferior_p (arg0).c_str (),
+	      target_debug_print_ptid_t (arg1).c_str (),
+	      target_debug_print_target_waitkind (arg2).c_str (),
+	      target_debug_print_bool (arg3).c_str (),
+	      target_debug_print_bool (arg4).c_str ());
 }
 
 void
@@ -1566,9 +1546,10 @@  debug_target::follow_clone (ptid_t arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->follow_clone (...)\n", this->beneath ()->shortname ());
   this->beneath ()->follow_clone (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->follow_clone (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->follow_clone (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str ());
 }
 
 int
@@ -1589,11 +1570,11 @@  debug_target::insert_exec_catchpoint (int arg0)
   gdb_printf (gdb_stdlog, "-> %s->insert_exec_catchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->insert_exec_catchpoint (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->insert_exec_catchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->insert_exec_catchpoint (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -1615,11 +1596,11 @@  debug_target::remove_exec_catchpoint (int arg0)
   gdb_printf (gdb_stdlog, "-> %s->remove_exec_catchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->remove_exec_catchpoint (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->remove_exec_catchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->remove_exec_catchpoint (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -1639,13 +1620,12 @@  debug_target::follow_exec (inferior *arg0, ptid_t arg1, const char *arg2)
 {
   gdb_printf (gdb_stdlog, "-> %s->follow_exec (...)\n", this->beneath ()->shortname ());
   this->beneath ()->follow_exec (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->follow_exec (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_inferior_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ptid_t (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_const_char_p (arg2), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->follow_exec (%s, %s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_inferior_p (arg0).c_str (),
+	      target_debug_print_ptid_t (arg1).c_str (),
+	      target_debug_print_const_char_p (arg2).c_str ());
 }
 
 int
@@ -1666,17 +1646,14 @@  debug_target::set_syscall_catchpoint (int arg0, bool arg1, int arg2, gdb::array_
   gdb_printf (gdb_stdlog, "-> %s->set_syscall_catchpoint (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->set_syscall_catchpoint (arg0, arg1, arg2, arg3);
-  gdb_printf (gdb_stdlog, "<- %s->set_syscall_catchpoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_bool (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg2), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_gdb_array_view_const_int (arg3), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->set_syscall_catchpoint (%s, %s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str (),
+	      target_debug_print_bool (arg1).c_str (),
+	      target_debug_print_int (arg2).c_str (),
+	      target_debug_print_gdb_array_view_const_int (arg3).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -1697,8 +1674,9 @@  debug_target::mourn_inferior ()
 {
   gdb_printf (gdb_stdlog, "-> %s->mourn_inferior (...)\n", this->beneath ()->shortname ());
   this->beneath ()->mourn_inferior ();
-  gdb_printf (gdb_stdlog, "<- %s->mourn_inferior (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->mourn_inferior ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -1717,9 +1695,10 @@  debug_target::pass_signals (gdb::array_view<const unsigned char> arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->pass_signals (...)\n", this->beneath ()->shortname ());
   this->beneath ()->pass_signals (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->pass_signals (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_signals (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->pass_signals (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_signals (arg0).c_str ());
 }
 
 void
@@ -1738,9 +1717,10 @@  debug_target::program_signals (gdb::array_view<const unsigned char> arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->program_signals (...)\n", this->beneath ()->shortname ());
   this->beneath ()->program_signals (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->program_signals (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_signals (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->program_signals (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_signals (arg0).c_str ());
 }
 
 bool
@@ -1761,11 +1741,11 @@  debug_target::thread_alive (ptid_t arg0)
   gdb_printf (gdb_stdlog, "-> %s->thread_alive (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->thread_alive (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->thread_alive (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->thread_alive (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -1785,8 +1765,9 @@  debug_target::update_thread_list ()
 {
   gdb_printf (gdb_stdlog, "-> %s->update_thread_list (...)\n", this->beneath ()->shortname ());
   this->beneath ()->update_thread_list ();
-  gdb_printf (gdb_stdlog, "<- %s->update_thread_list (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->update_thread_list ()\n",
+	      this->beneath ()->shortname ());
 }
 
 std::string
@@ -1807,11 +1788,11 @@  debug_target::pid_to_str (ptid_t arg0)
   gdb_printf (gdb_stdlog, "-> %s->pid_to_str (...)\n", this->beneath ()->shortname ());
   std::string result
     = this->beneath ()->pid_to_str (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->pid_to_str (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_std_string (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->pid_to_str (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str (),
+	      target_debug_print_std_string (result).c_str ());
   return result;
 }
 
@@ -1833,11 +1814,11 @@  debug_target::extra_thread_info (thread_info *arg0)
   gdb_printf (gdb_stdlog, "-> %s->extra_thread_info (...)\n", this->beneath ()->shortname ());
   const char * result
     = this->beneath ()->extra_thread_info (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->extra_thread_info (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_thread_info_p (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_const_char_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->extra_thread_info (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_thread_info_p (arg0).c_str (),
+	      target_debug_print_const_char_p (result).c_str ());
   return result;
 }
 
@@ -1859,11 +1840,11 @@  debug_target::thread_name (thread_info *arg0)
   gdb_printf (gdb_stdlog, "-> %s->thread_name (...)\n", this->beneath ()->shortname ());
   const char * result
     = this->beneath ()->thread_name (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->thread_name (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_thread_info_p (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_const_char_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->thread_name (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_thread_info_p (arg0).c_str (),
+	      target_debug_print_const_char_p (result).c_str ());
   return result;
 }
 
@@ -1885,15 +1866,13 @@  debug_target::thread_handle_to_thread_info (const gdb_byte *arg0, int arg1, infe
   gdb_printf (gdb_stdlog, "-> %s->thread_handle_to_thread_info (...)\n", this->beneath ()->shortname ());
   thread_info * result
     = this->beneath ()->thread_handle_to_thread_info (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->thread_handle_to_thread_info (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_gdb_byte_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_inferior_p (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_thread_info_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->thread_handle_to_thread_info (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_gdb_byte_p (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str (),
+	      target_debug_print_inferior_p (arg2).c_str (),
+	      target_debug_print_thread_info_p (result).c_str ());
   return result;
 }
 
@@ -1915,11 +1894,11 @@  debug_target::thread_info_to_thread_handle (struct thread_info *arg0)
   gdb_printf (gdb_stdlog, "-> %s->thread_info_to_thread_handle (...)\n", this->beneath ()->shortname ());
   gdb::array_view<const_gdb_byte> result
     = this->beneath ()->thread_info_to_thread_handle (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->thread_info_to_thread_handle (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_thread_info_p (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_gdb_array_view_const_gdb_byte (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->thread_info_to_thread_handle (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_thread_info_p (arg0).c_str (),
+	      target_debug_print_gdb_array_view_const_gdb_byte (result).c_str ());
   return result;
 }
 
@@ -1939,9 +1918,10 @@  debug_target::stop (ptid_t arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->stop (...)\n", this->beneath ()->shortname ());
   this->beneath ()->stop (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->stop (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->stop (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str ());
 }
 
 void
@@ -1960,8 +1940,9 @@  debug_target::interrupt ()
 {
   gdb_printf (gdb_stdlog, "-> %s->interrupt (...)\n", this->beneath ()->shortname ());
   this->beneath ()->interrupt ();
-  gdb_printf (gdb_stdlog, "<- %s->interrupt (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->interrupt ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -1981,8 +1962,9 @@  debug_target::pass_ctrlc ()
 {
   gdb_printf (gdb_stdlog, "-> %s->pass_ctrlc (...)\n", this->beneath ()->shortname ());
   this->beneath ()->pass_ctrlc ();
-  gdb_printf (gdb_stdlog, "<- %s->pass_ctrlc (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->pass_ctrlc ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -2002,11 +1984,11 @@  debug_target::rcmd (const char *arg0, struct ui_file *arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->rcmd (...)\n", this->beneath ()->shortname ());
   this->beneath ()->rcmd (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->rcmd (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_char_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ui_file_p (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->rcmd (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_char_p (arg0).c_str (),
+	      target_debug_print_ui_file_p (arg1).c_str ());
 }
 
 const char *
@@ -2027,11 +2009,11 @@  debug_target::pid_to_exec_file (int arg0)
   gdb_printf (gdb_stdlog, "-> %s->pid_to_exec_file (...)\n", this->beneath ()->shortname ());
   const char * result
     = this->beneath ()->pid_to_exec_file (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->pid_to_exec_file (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_const_char_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->pid_to_exec_file (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str (),
+	      target_debug_print_const_char_p (result).c_str ());
   return result;
 }
 
@@ -2051,9 +2033,10 @@  debug_target::log_command (const char *arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->log_command (...)\n", this->beneath ()->shortname ());
   this->beneath ()->log_command (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->log_command (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_char_p (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->log_command (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_char_p (arg0).c_str ());
 }
 
 const std::vector<target_section> *
@@ -2074,10 +2057,10 @@  debug_target::get_section_table ()
   gdb_printf (gdb_stdlog, "-> %s->get_section_table (...)\n", this->beneath ()->shortname ());
   const std::vector<target_section> * result
     = this->beneath ()->get_section_table ();
-  gdb_printf (gdb_stdlog, "<- %s->get_section_table (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_const_std_vector_target_section_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_section_table () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_std_vector_target_section_p (result).c_str ());
   return result;
 }
 
@@ -2099,10 +2082,10 @@  debug_target::get_thread_control_capabilities ()
   gdb_printf (gdb_stdlog, "-> %s->get_thread_control_capabilities (...)\n", this->beneath ()->shortname ());
   thread_control_capabilities result
     = this->beneath ()->get_thread_control_capabilities ();
-  gdb_printf (gdb_stdlog, "<- %s->get_thread_control_capabilities (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_thread_control_capabilities (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_thread_control_capabilities () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_thread_control_capabilities (result).c_str ());
   return result;
 }
 
@@ -2124,10 +2107,10 @@  debug_target::attach_no_wait ()
   gdb_printf (gdb_stdlog, "-> %s->attach_no_wait (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->attach_no_wait ();
-  gdb_printf (gdb_stdlog, "<- %s->attach_no_wait (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->attach_no_wait () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2149,10 +2132,10 @@  debug_target::can_async_p ()
   gdb_printf (gdb_stdlog, "-> %s->can_async_p (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->can_async_p ();
-  gdb_printf (gdb_stdlog, "<- %s->can_async_p (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->can_async_p () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2174,10 +2157,10 @@  debug_target::is_async_p ()
   gdb_printf (gdb_stdlog, "-> %s->is_async_p (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->is_async_p ();
-  gdb_printf (gdb_stdlog, "<- %s->is_async_p (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->is_async_p () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2198,9 +2181,10 @@  debug_target::async (bool arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->async (...)\n", this->beneath ()->shortname ());
   this->beneath ()->async (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->async (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_bool (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->async (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (arg0).c_str ());
 }
 
 int
@@ -2221,10 +2205,10 @@  debug_target::async_wait_fd ()
   gdb_printf (gdb_stdlog, "-> %s->async_wait_fd (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->async_wait_fd ();
-  gdb_printf (gdb_stdlog, "<- %s->async_wait_fd (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->async_wait_fd () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -2246,10 +2230,10 @@  debug_target::has_pending_events ()
   gdb_printf (gdb_stdlog, "-> %s->has_pending_events (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->has_pending_events ();
-  gdb_printf (gdb_stdlog, "<- %s->has_pending_events (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->has_pending_events () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2269,9 +2253,10 @@  debug_target::thread_events (int arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->thread_events (...)\n", this->beneath ()->shortname ());
   this->beneath ()->thread_events (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->thread_events (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->thread_events (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str ());
 }
 
 bool
@@ -2292,11 +2277,11 @@  debug_target::supports_set_thread_options (gdb_thread_options arg0)
   gdb_printf (gdb_stdlog, "-> %s->supports_set_thread_options (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_set_thread_options (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->supports_set_thread_options (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_gdb_thread_options (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_set_thread_options (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_gdb_thread_options (arg0).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2318,10 +2303,10 @@  debug_target::supports_non_stop ()
   gdb_printf (gdb_stdlog, "-> %s->supports_non_stop (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_non_stop ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_non_stop (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_non_stop () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2343,10 +2328,10 @@  debug_target::always_non_stop_p ()
   gdb_printf (gdb_stdlog, "-> %s->always_non_stop_p (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->always_non_stop_p ();
-  gdb_printf (gdb_stdlog, "<- %s->always_non_stop_p (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->always_non_stop_p () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2368,13 +2353,12 @@  debug_target::find_memory_regions (find_memory_region_ftype arg0, void *arg1)
   gdb_printf (gdb_stdlog, "-> %s->find_memory_regions (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->find_memory_regions (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->find_memory_regions (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_find_memory_region_ftype (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_void_p (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->find_memory_regions (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_find_memory_region_ftype (arg0).c_str (),
+	      target_debug_print_void_p (arg1).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -2396,13 +2380,12 @@  debug_target::make_corefile_notes (bfd *arg0, int *arg1)
   gdb_printf (gdb_stdlog, "-> %s->make_corefile_notes (...)\n", this->beneath ()->shortname ());
   gdb::unique_xmalloc_ptr<char> result
     = this->beneath ()->make_corefile_notes (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->make_corefile_notes (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_bfd_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int_p (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_gdb_unique_xmalloc_ptr_char (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->make_corefile_notes (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bfd_p (arg0).c_str (),
+	      target_debug_print_int_p (arg1).c_str (),
+	      target_debug_print_gdb_unique_xmalloc_ptr_char (result).c_str ());
   return result;
 }
 
@@ -2424,13 +2407,12 @@  debug_target::get_bookmark (const char *arg0, int arg1)
   gdb_printf (gdb_stdlog, "-> %s->get_bookmark (...)\n", this->beneath ()->shortname ());
   gdb_byte * result
     = this->beneath ()->get_bookmark (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->get_bookmark (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_char_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_gdb_byte_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_bookmark (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_char_p (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str (),
+	      target_debug_print_gdb_byte_p (result).c_str ());
   return result;
 }
 
@@ -2451,11 +2433,11 @@  debug_target::goto_bookmark (const gdb_byte *arg0, int arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->goto_bookmark (...)\n", this->beneath ()->shortname ());
   this->beneath ()->goto_bookmark (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->goto_bookmark (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_gdb_byte_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->goto_bookmark (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_gdb_byte_p (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str ());
 }
 
 CORE_ADDR
@@ -2476,15 +2458,13 @@  debug_target::get_thread_local_address (ptid_t arg0, CORE_ADDR arg1, CORE_ADDR a
   gdb_printf (gdb_stdlog, "-> %s->get_thread_local_address (...)\n", this->beneath ()->shortname ());
   CORE_ADDR result
     = this->beneath ()->get_thread_local_address (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->get_thread_local_address (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_CORE_ADDR (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_thread_local_address (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str (),
+	      target_debug_print_CORE_ADDR (arg1).c_str (),
+	      target_debug_print_CORE_ADDR (arg2).c_str (),
+	      target_debug_print_CORE_ADDR (result).c_str ());
   return result;
 }
 
@@ -2506,23 +2486,17 @@  debug_target::xfer_partial (enum target_object arg0, const char *arg1, gdb_byte
   gdb_printf (gdb_stdlog, "-> %s->xfer_partial (...)\n", this->beneath ()->shortname ());
   enum target_xfer_status result
     = this->beneath ()->xfer_partial (arg0, arg1, arg2, arg3, arg4, arg5, arg6);
-  gdb_printf (gdb_stdlog, "<- %s->xfer_partial (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_target_object (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_const_char_p (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_gdb_byte_p (arg2), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_const_gdb_byte_p (arg3), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ULONGEST (arg4), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ULONGEST (arg5), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ULONGEST_p (arg6), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_target_xfer_status (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->xfer_partial (%s, %s, %s, %s, %s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_target_object (arg0).c_str (),
+	      target_debug_print_const_char_p (arg1).c_str (),
+	      target_debug_print_gdb_byte_p (arg2).c_str (),
+	      target_debug_print_const_gdb_byte_p (arg3).c_str (),
+	      target_debug_print_ULONGEST (arg4).c_str (),
+	      target_debug_print_ULONGEST (arg5).c_str (),
+	      target_debug_print_ULONGEST_p (arg6).c_str (),
+	      target_debug_print_target_xfer_status (result).c_str ());
   return result;
 }
 
@@ -2544,10 +2518,10 @@  debug_target::get_memory_xfer_limit ()
   gdb_printf (gdb_stdlog, "-> %s->get_memory_xfer_limit (...)\n", this->beneath ()->shortname ());
   ULONGEST result
     = this->beneath ()->get_memory_xfer_limit ();
-  gdb_printf (gdb_stdlog, "<- %s->get_memory_xfer_limit (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_ULONGEST (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_memory_xfer_limit () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ULONGEST (result).c_str ());
   return result;
 }
 
@@ -2569,10 +2543,10 @@  debug_target::memory_map ()
   gdb_printf (gdb_stdlog, "-> %s->memory_map (...)\n", this->beneath ()->shortname ());
   std::vector<mem_region> result
     = this->beneath ()->memory_map ();
-  gdb_printf (gdb_stdlog, "<- %s->memory_map (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_std_vector_mem_region (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->memory_map () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_std_vector_mem_region (result).c_str ());
   return result;
 }
 
@@ -2593,11 +2567,11 @@  debug_target::flash_erase (ULONGEST arg0, LONGEST arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->flash_erase (...)\n", this->beneath ()->shortname ());
   this->beneath ()->flash_erase (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->flash_erase (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ULONGEST (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_LONGEST (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->flash_erase (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ULONGEST (arg0).c_str (),
+	      target_debug_print_LONGEST (arg1).c_str ());
 }
 
 void
@@ -2617,8 +2591,9 @@  debug_target::flash_done ()
 {
   gdb_printf (gdb_stdlog, "-> %s->flash_done (...)\n", this->beneath ()->shortname ());
   this->beneath ()->flash_done ();
-  gdb_printf (gdb_stdlog, "<- %s->flash_done (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->flash_done ()\n",
+	      this->beneath ()->shortname ());
 }
 
 const struct target_desc *
@@ -2639,10 +2614,10 @@  debug_target::read_description ()
   gdb_printf (gdb_stdlog, "-> %s->read_description (...)\n", this->beneath ()->shortname ());
   const struct target_desc * result
     = this->beneath ()->read_description ();
-  gdb_printf (gdb_stdlog, "<- %s->read_description (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_const_target_desc_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->read_description () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_target_desc_p (result).c_str ());
   return result;
 }
 
@@ -2664,13 +2639,12 @@  debug_target::get_ada_task_ptid (long arg0, ULONGEST arg1)
   gdb_printf (gdb_stdlog, "-> %s->get_ada_task_ptid (...)\n", this->beneath ()->shortname ());
   ptid_t result
     = this->beneath ()->get_ada_task_ptid (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->get_ada_task_ptid (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_long (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ULONGEST (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_ptid_t (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_ada_task_ptid (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_long (arg0).c_str (),
+	      target_debug_print_ULONGEST (arg1).c_str (),
+	      target_debug_print_ptid_t (result).c_str ());
   return result;
 }
 
@@ -2692,17 +2666,14 @@  debug_target::auxv_parse (const gdb_byte **arg0, const gdb_byte *arg1, CORE_ADDR
   gdb_printf (gdb_stdlog, "-> %s->auxv_parse (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->auxv_parse (arg0, arg1, arg2, arg3);
-  gdb_printf (gdb_stdlog, "<- %s->auxv_parse (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_gdb_byte_pp (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_const_gdb_byte_p (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR_p (arg2), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR_p (arg3), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->auxv_parse (%s, %s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_gdb_byte_pp (arg0).c_str (),
+	      target_debug_print_const_gdb_byte_p (arg1).c_str (),
+	      target_debug_print_CORE_ADDR_p (arg2).c_str (),
+	      target_debug_print_CORE_ADDR_p (arg3).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -2724,19 +2695,15 @@  debug_target::search_memory (CORE_ADDR arg0, ULONGEST arg1, const gdb_byte *arg2
   gdb_printf (gdb_stdlog, "-> %s->search_memory (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->search_memory (arg0, arg1, arg2, arg3, arg4);
-  gdb_printf (gdb_stdlog, "<- %s->search_memory (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ULONGEST (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_const_gdb_byte_p (arg2), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ULONGEST (arg3), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR_p (arg4), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->search_memory (%s, %s, %s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_ULONGEST (arg1).c_str (),
+	      target_debug_print_const_gdb_byte_p (arg2).c_str (),
+	      target_debug_print_ULONGEST (arg3).c_str (),
+	      target_debug_print_CORE_ADDR_p (arg4).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -2758,10 +2725,10 @@  debug_target::can_execute_reverse ()
   gdb_printf (gdb_stdlog, "-> %s->can_execute_reverse (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->can_execute_reverse ();
-  gdb_printf (gdb_stdlog, "<- %s->can_execute_reverse (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->can_execute_reverse () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2783,10 +2750,10 @@  debug_target::execution_direction ()
   gdb_printf (gdb_stdlog, "-> %s->execution_direction (...)\n", this->beneath ()->shortname ());
   enum exec_direction_kind result
     = this->beneath ()->execution_direction ();
-  gdb_printf (gdb_stdlog, "<- %s->execution_direction (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_exec_direction_kind (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->execution_direction () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_exec_direction_kind (result).c_str ());
   return result;
 }
 
@@ -2808,10 +2775,10 @@  debug_target::supports_multi_process ()
   gdb_printf (gdb_stdlog, "-> %s->supports_multi_process (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_multi_process ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_multi_process (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_multi_process () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2833,10 +2800,10 @@  debug_target::supports_enable_disable_tracepoint ()
   gdb_printf (gdb_stdlog, "-> %s->supports_enable_disable_tracepoint (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_enable_disable_tracepoint ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_enable_disable_tracepoint (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_enable_disable_tracepoint () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2858,10 +2825,10 @@  debug_target::supports_disable_randomization ()
   gdb_printf (gdb_stdlog, "-> %s->supports_disable_randomization (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_disable_randomization ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_disable_randomization (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_disable_randomization () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2883,10 +2850,10 @@  debug_target::supports_string_tracing ()
   gdb_printf (gdb_stdlog, "-> %s->supports_string_tracing (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_string_tracing ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_string_tracing (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_string_tracing () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2908,10 +2875,10 @@  debug_target::supports_evaluation_of_breakpoint_conditions ()
   gdb_printf (gdb_stdlog, "-> %s->supports_evaluation_of_breakpoint_conditions (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_evaluation_of_breakpoint_conditions ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_evaluation_of_breakpoint_conditions (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_evaluation_of_breakpoint_conditions () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2933,10 +2900,10 @@  debug_target::supports_dumpcore ()
   gdb_printf (gdb_stdlog, "-> %s->supports_dumpcore (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_dumpcore ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_dumpcore (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_dumpcore () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -2956,9 +2923,10 @@  debug_target::dumpcore (const char *arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->dumpcore (...)\n", this->beneath ()->shortname ());
   this->beneath ()->dumpcore (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->dumpcore (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_char_p (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->dumpcore (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_char_p (arg0).c_str ());
 }
 
 bool
@@ -2979,10 +2947,10 @@  debug_target::can_run_breakpoint_commands ()
   gdb_printf (gdb_stdlog, "-> %s->can_run_breakpoint_commands (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->can_run_breakpoint_commands ();
-  gdb_printf (gdb_stdlog, "<- %s->can_run_breakpoint_commands (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->can_run_breakpoint_commands () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -3004,11 +2972,11 @@  debug_target::thread_architecture (ptid_t arg0)
   gdb_printf (gdb_stdlog, "-> %s->thread_architecture (...)\n", this->beneath ()->shortname ());
   struct gdbarch * result
     = this->beneath ()->thread_architecture (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->thread_architecture (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_gdbarch_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->thread_architecture (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str (),
+	      target_debug_print_gdbarch_p (result).c_str ());
   return result;
 }
 
@@ -3030,10 +2998,10 @@  debug_target::filesystem_is_local ()
   gdb_printf (gdb_stdlog, "-> %s->filesystem_is_local (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->filesystem_is_local ();
-  gdb_printf (gdb_stdlog, "<- %s->filesystem_is_local (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->filesystem_is_local () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -3054,8 +3022,9 @@  debug_target::trace_init ()
 {
   gdb_printf (gdb_stdlog, "-> %s->trace_init (...)\n", this->beneath ()->shortname ());
   this->beneath ()->trace_init ();
-  gdb_printf (gdb_stdlog, "<- %s->trace_init (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->trace_init ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -3075,9 +3044,10 @@  debug_target::download_tracepoint (struct bp_location *arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->download_tracepoint (...)\n", this->beneath ()->shortname ());
   this->beneath ()->download_tracepoint (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->download_tracepoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_bp_location_p (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->download_tracepoint (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bp_location_p (arg0).c_str ());
 }
 
 bool
@@ -3098,10 +3068,10 @@  debug_target::can_download_tracepoint ()
   gdb_printf (gdb_stdlog, "-> %s->can_download_tracepoint (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->can_download_tracepoint ();
-  gdb_printf (gdb_stdlog, "<- %s->can_download_tracepoint (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->can_download_tracepoint () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -3122,9 +3092,10 @@  debug_target::download_trace_state_variable (const trace_state_variable &arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->download_trace_state_variable (...)\n", this->beneath ()->shortname ());
   this->beneath ()->download_trace_state_variable (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->download_trace_state_variable (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_trace_state_variable_r (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->download_trace_state_variable (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_trace_state_variable_r (arg0).c_str ());
 }
 
 void
@@ -3144,9 +3115,10 @@  debug_target::enable_tracepoint (struct bp_location *arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->enable_tracepoint (...)\n", this->beneath ()->shortname ());
   this->beneath ()->enable_tracepoint (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->enable_tracepoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_bp_location_p (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->enable_tracepoint (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bp_location_p (arg0).c_str ());
 }
 
 void
@@ -3166,9 +3138,10 @@  debug_target::disable_tracepoint (struct bp_location *arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->disable_tracepoint (...)\n", this->beneath ()->shortname ());
   this->beneath ()->disable_tracepoint (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->disable_tracepoint (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_bp_location_p (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->disable_tracepoint (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bp_location_p (arg0).c_str ());
 }
 
 void
@@ -3188,8 +3161,9 @@  debug_target::trace_set_readonly_regions ()
 {
   gdb_printf (gdb_stdlog, "-> %s->trace_set_readonly_regions (...)\n", this->beneath ()->shortname ());
   this->beneath ()->trace_set_readonly_regions ();
-  gdb_printf (gdb_stdlog, "<- %s->trace_set_readonly_regions (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->trace_set_readonly_regions ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -3209,8 +3183,9 @@  debug_target::trace_start ()
 {
   gdb_printf (gdb_stdlog, "-> %s->trace_start (...)\n", this->beneath ()->shortname ());
   this->beneath ()->trace_start ();
-  gdb_printf (gdb_stdlog, "<- %s->trace_start (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->trace_start ()\n",
+	      this->beneath ()->shortname ());
 }
 
 int
@@ -3231,11 +3206,11 @@  debug_target::get_trace_status (struct trace_status *arg0)
   gdb_printf (gdb_stdlog, "-> %s->get_trace_status (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->get_trace_status (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->get_trace_status (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_trace_status_p (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_trace_status (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_trace_status_p (arg0).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -3256,11 +3231,11 @@  debug_target::get_tracepoint_status (tracepoint *arg0, struct uploaded_tp *arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->get_tracepoint_status (...)\n", this->beneath ()->shortname ());
   this->beneath ()->get_tracepoint_status (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->get_tracepoint_status (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_tracepoint_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_uploaded_tp_p (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_tracepoint_status (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_tracepoint_p (arg0).c_str (),
+	      target_debug_print_uploaded_tp_p (arg1).c_str ());
 }
 
 void
@@ -3280,8 +3255,9 @@  debug_target::trace_stop ()
 {
   gdb_printf (gdb_stdlog, "-> %s->trace_stop (...)\n", this->beneath ()->shortname ());
   this->beneath ()->trace_stop ();
-  gdb_printf (gdb_stdlog, "<- %s->trace_stop (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->trace_stop ()\n",
+	      this->beneath ()->shortname ());
 }
 
 int
@@ -3302,19 +3278,15 @@  debug_target::trace_find (enum trace_find_type arg0, int arg1, CORE_ADDR arg2, C
   gdb_printf (gdb_stdlog, "-> %s->trace_find (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->trace_find (arg0, arg1, arg2, arg3, arg4);
-  gdb_printf (gdb_stdlog, "<- %s->trace_find (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_trace_find_type (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR (arg2), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR (arg3), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int_p (arg4), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->trace_find (%s, %s, %s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_trace_find_type (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str (),
+	      target_debug_print_CORE_ADDR (arg2).c_str (),
+	      target_debug_print_CORE_ADDR (arg3).c_str (),
+	      target_debug_print_int_p (arg4).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -3336,13 +3308,12 @@  debug_target::get_trace_state_variable_value (int arg0, LONGEST *arg1)
   gdb_printf (gdb_stdlog, "-> %s->get_trace_state_variable_value (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->get_trace_state_variable_value (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->get_trace_state_variable_value (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_LONGEST_p (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_trace_state_variable_value (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str (),
+	      target_debug_print_LONGEST_p (arg1).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -3364,11 +3335,11 @@  debug_target::save_trace_data (const char *arg0)
   gdb_printf (gdb_stdlog, "-> %s->save_trace_data (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->save_trace_data (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->save_trace_data (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_char_p (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->save_trace_data (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_char_p (arg0).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -3390,11 +3361,11 @@  debug_target::upload_tracepoints (struct uploaded_tp **arg0)
   gdb_printf (gdb_stdlog, "-> %s->upload_tracepoints (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->upload_tracepoints (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->upload_tracepoints (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_uploaded_tp_pp (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->upload_tracepoints (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_uploaded_tp_pp (arg0).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -3416,11 +3387,11 @@  debug_target::upload_trace_state_variables (struct uploaded_tsv **arg0)
   gdb_printf (gdb_stdlog, "-> %s->upload_trace_state_variables (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->upload_trace_state_variables (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->upload_trace_state_variables (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_uploaded_tsv_pp (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->upload_trace_state_variables (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_uploaded_tsv_pp (arg0).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -3442,15 +3413,13 @@  debug_target::get_raw_trace_data (gdb_byte *arg0, ULONGEST arg1, LONGEST arg2)
   gdb_printf (gdb_stdlog, "-> %s->get_raw_trace_data (...)\n", this->beneath ()->shortname ());
   LONGEST result
     = this->beneath ()->get_raw_trace_data (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->get_raw_trace_data (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_gdb_byte_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ULONGEST (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_LONGEST (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_LONGEST (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_raw_trace_data (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_gdb_byte_p (arg0).c_str (),
+	      target_debug_print_ULONGEST (arg1).c_str (),
+	      target_debug_print_LONGEST (arg2).c_str (),
+	      target_debug_print_LONGEST (result).c_str ());
   return result;
 }
 
@@ -3472,10 +3441,10 @@  debug_target::get_min_fast_tracepoint_insn_len ()
   gdb_printf (gdb_stdlog, "-> %s->get_min_fast_tracepoint_insn_len (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->get_min_fast_tracepoint_insn_len ();
-  gdb_printf (gdb_stdlog, "<- %s->get_min_fast_tracepoint_insn_len (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_min_fast_tracepoint_insn_len () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -3495,9 +3464,10 @@  debug_target::set_disconnected_tracing (int arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->set_disconnected_tracing (...)\n", this->beneath ()->shortname ());
   this->beneath ()->set_disconnected_tracing (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->set_disconnected_tracing (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->set_disconnected_tracing (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str ());
 }
 
 void
@@ -3516,9 +3486,10 @@  debug_target::set_circular_trace_buffer (int arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->set_circular_trace_buffer (...)\n", this->beneath ()->shortname ());
   this->beneath ()->set_circular_trace_buffer (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->set_circular_trace_buffer (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->set_circular_trace_buffer (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str ());
 }
 
 void
@@ -3537,9 +3508,10 @@  debug_target::set_trace_buffer_size (LONGEST arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->set_trace_buffer_size (...)\n", this->beneath ()->shortname ());
   this->beneath ()->set_trace_buffer_size (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->set_trace_buffer_size (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_LONGEST (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->set_trace_buffer_size (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_LONGEST (arg0).c_str ());
 }
 
 bool
@@ -3560,15 +3532,13 @@  debug_target::set_trace_notes (const char *arg0, const char *arg1, const char *a
   gdb_printf (gdb_stdlog, "-> %s->set_trace_notes (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->set_trace_notes (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->set_trace_notes (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_char_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_const_char_p (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_const_char_p (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->set_trace_notes (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_char_p (arg0).c_str (),
+	      target_debug_print_const_char_p (arg1).c_str (),
+	      target_debug_print_const_char_p (arg2).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -3590,11 +3560,11 @@  debug_target::core_of_thread (ptid_t arg0)
   gdb_printf (gdb_stdlog, "-> %s->core_of_thread (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->core_of_thread (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->core_of_thread (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->core_of_thread (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -3616,15 +3586,13 @@  debug_target::verify_memory (const gdb_byte *arg0, CORE_ADDR arg1, ULONGEST arg2
   gdb_printf (gdb_stdlog, "-> %s->verify_memory (...)\n", this->beneath ()->shortname ());
   int result
     = this->beneath ()->verify_memory (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->verify_memory (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_gdb_byte_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ULONGEST (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_int (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->verify_memory (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_gdb_byte_p (arg0).c_str (),
+	      target_debug_print_CORE_ADDR (arg1).c_str (),
+	      target_debug_print_ULONGEST (arg2).c_str (),
+	      target_debug_print_int (result).c_str ());
   return result;
 }
 
@@ -3646,13 +3614,12 @@  debug_target::get_tib_address (ptid_t arg0, CORE_ADDR *arg1)
   gdb_printf (gdb_stdlog, "-> %s->get_tib_address (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->get_tib_address (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->get_tib_address (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_CORE_ADDR_p (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_tib_address (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str (),
+	      target_debug_print_CORE_ADDR_p (arg1).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -3672,8 +3639,9 @@  debug_target::set_permissions ()
 {
   gdb_printf (gdb_stdlog, "-> %s->set_permissions (...)\n", this->beneath ()->shortname ());
   this->beneath ()->set_permissions ();
-  gdb_printf (gdb_stdlog, "<- %s->set_permissions (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->set_permissions ()\n",
+	      this->beneath ()->shortname ());
 }
 
 bool
@@ -3694,13 +3662,12 @@  debug_target::static_tracepoint_marker_at (CORE_ADDR arg0, static_tracepoint_mar
   gdb_printf (gdb_stdlog, "-> %s->static_tracepoint_marker_at (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->static_tracepoint_marker_at (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->static_tracepoint_marker_at (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_static_tracepoint_marker_p (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->static_tracepoint_marker_at (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_static_tracepoint_marker_p (arg1).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -3722,11 +3689,11 @@  debug_target::static_tracepoint_markers_by_strid (const char *arg0)
   gdb_printf (gdb_stdlog, "-> %s->static_tracepoint_markers_by_strid (...)\n", this->beneath ()->shortname ());
   std::vector<static_tracepoint_marker> result
     = this->beneath ()->static_tracepoint_markers_by_strid (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->static_tracepoint_markers_by_strid (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_char_p (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_std_vector_static_tracepoint_marker (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->static_tracepoint_markers_by_strid (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_char_p (arg0).c_str (),
+	      target_debug_print_std_vector_static_tracepoint_marker (result).c_str ());
   return result;
 }
 
@@ -3748,10 +3715,10 @@  debug_target::traceframe_info ()
   gdb_printf (gdb_stdlog, "-> %s->traceframe_info (...)\n", this->beneath ()->shortname ());
   traceframe_info_up result
     = this->beneath ()->traceframe_info ();
-  gdb_printf (gdb_stdlog, "<- %s->traceframe_info (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_traceframe_info_up (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->traceframe_info () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_traceframe_info_up (result).c_str ());
   return result;
 }
 
@@ -3773,11 +3740,11 @@  debug_target::use_agent (bool arg0)
   gdb_printf (gdb_stdlog, "-> %s->use_agent (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->use_agent (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->use_agent (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_bool (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->use_agent (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (arg0).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -3799,10 +3766,10 @@  debug_target::can_use_agent ()
   gdb_printf (gdb_stdlog, "-> %s->can_use_agent (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->can_use_agent ();
-  gdb_printf (gdb_stdlog, "<- %s->can_use_agent (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->can_use_agent () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -3824,13 +3791,12 @@  debug_target::enable_btrace (thread_info *arg0, const struct btrace_config *arg1
   gdb_printf (gdb_stdlog, "-> %s->enable_btrace (...)\n", this->beneath ()->shortname ());
   struct btrace_target_info * result
     = this->beneath ()->enable_btrace (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->enable_btrace (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_thread_info_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_const_btrace_config_p (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_btrace_target_info_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->enable_btrace (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_thread_info_p (arg0).c_str (),
+	      target_debug_print_const_btrace_config_p (arg1).c_str (),
+	      target_debug_print_btrace_target_info_p (result).c_str ());
   return result;
 }
 
@@ -3851,9 +3817,10 @@  debug_target::disable_btrace (struct btrace_target_info *arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->disable_btrace (...)\n", this->beneath ()->shortname ());
   this->beneath ()->disable_btrace (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->disable_btrace (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_btrace_target_info_p (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->disable_btrace (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_btrace_target_info_p (arg0).c_str ());
 }
 
 void
@@ -3873,9 +3840,10 @@  debug_target::teardown_btrace (struct btrace_target_info *arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->teardown_btrace (...)\n", this->beneath ()->shortname ());
   this->beneath ()->teardown_btrace (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->teardown_btrace (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_btrace_target_info_p (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->teardown_btrace (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_btrace_target_info_p (arg0).c_str ());
 }
 
 enum btrace_error
@@ -3896,15 +3864,13 @@  debug_target::read_btrace (struct btrace_data *arg0, struct btrace_target_info *
   gdb_printf (gdb_stdlog, "-> %s->read_btrace (...)\n", this->beneath ()->shortname ());
   enum btrace_error result
     = this->beneath ()->read_btrace (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->read_btrace (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_btrace_data_p (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_btrace_target_info_p (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_btrace_read_type (arg2), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_btrace_error (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->read_btrace (%s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_btrace_data_p (arg0).c_str (),
+	      target_debug_print_btrace_target_info_p (arg1).c_str (),
+	      target_debug_print_btrace_read_type (arg2).c_str (),
+	      target_debug_print_btrace_error (result).c_str ());
   return result;
 }
 
@@ -3926,11 +3892,11 @@  debug_target::btrace_conf (const struct btrace_target_info *arg0)
   gdb_printf (gdb_stdlog, "-> %s->btrace_conf (...)\n", this->beneath ()->shortname ());
   const struct btrace_config * result
     = this->beneath ()->btrace_conf (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->btrace_conf (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_btrace_target_info_p (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_const_btrace_config_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->btrace_conf (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_btrace_target_info_p (arg0).c_str (),
+	      target_debug_print_const_btrace_config_p (result).c_str ());
   return result;
 }
 
@@ -3952,11 +3918,11 @@  debug_target::record_method (ptid_t arg0)
   gdb_printf (gdb_stdlog, "-> %s->record_method (...)\n", this->beneath ()->shortname ());
   enum record_method result
     = this->beneath ()->record_method (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->record_method (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_record_method (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->record_method (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str (),
+	      target_debug_print_record_method (result).c_str ());
   return result;
 }
 
@@ -3976,8 +3942,9 @@  debug_target::stop_recording ()
 {
   gdb_printf (gdb_stdlog, "-> %s->stop_recording (...)\n", this->beneath ()->shortname ());
   this->beneath ()->stop_recording ();
-  gdb_printf (gdb_stdlog, "<- %s->stop_recording (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->stop_recording ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -3996,8 +3963,9 @@  debug_target::info_record ()
 {
   gdb_printf (gdb_stdlog, "-> %s->info_record (...)\n", this->beneath ()->shortname ());
   this->beneath ()->info_record ();
-  gdb_printf (gdb_stdlog, "<- %s->info_record (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->info_record ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -4017,9 +3985,10 @@  debug_target::save_record (const char *arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->save_record (...)\n", this->beneath ()->shortname ());
   this->beneath ()->save_record (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->save_record (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_const_char_p (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->save_record (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_char_p (arg0).c_str ());
 }
 
 bool
@@ -4040,10 +4009,10 @@  debug_target::supports_delete_record ()
   gdb_printf (gdb_stdlog, "-> %s->supports_delete_record (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_delete_record ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_delete_record (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_delete_record () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -4064,8 +4033,9 @@  debug_target::delete_record ()
 {
   gdb_printf (gdb_stdlog, "-> %s->delete_record (...)\n", this->beneath ()->shortname ());
   this->beneath ()->delete_record ();
-  gdb_printf (gdb_stdlog, "<- %s->delete_record (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->delete_record ()\n",
+	      this->beneath ()->shortname ());
 }
 
 bool
@@ -4086,11 +4056,11 @@  debug_target::record_is_replaying (ptid_t arg0)
   gdb_printf (gdb_stdlog, "-> %s->record_is_replaying (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->record_is_replaying (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->record_is_replaying (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->record_is_replaying (%s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -4112,13 +4082,12 @@  debug_target::record_will_replay (ptid_t arg0, int arg1)
   gdb_printf (gdb_stdlog, "-> %s->record_will_replay (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->record_will_replay (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->record_will_replay (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ptid_t (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->record_will_replay (%s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ptid_t (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -4138,8 +4107,9 @@  debug_target::record_stop_replaying ()
 {
   gdb_printf (gdb_stdlog, "-> %s->record_stop_replaying (...)\n", this->beneath ()->shortname ());
   this->beneath ()->record_stop_replaying ();
-  gdb_printf (gdb_stdlog, "<- %s->record_stop_replaying (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->record_stop_replaying ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -4159,8 +4129,9 @@  debug_target::goto_record_begin ()
 {
   gdb_printf (gdb_stdlog, "-> %s->goto_record_begin (...)\n", this->beneath ()->shortname ());
   this->beneath ()->goto_record_begin ();
-  gdb_printf (gdb_stdlog, "<- %s->goto_record_begin (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->goto_record_begin ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -4180,8 +4151,9 @@  debug_target::goto_record_end ()
 {
   gdb_printf (gdb_stdlog, "-> %s->goto_record_end (...)\n", this->beneath ()->shortname ());
   this->beneath ()->goto_record_end ();
-  gdb_printf (gdb_stdlog, "<- %s->goto_record_end (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->goto_record_end ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -4201,9 +4173,10 @@  debug_target::goto_record (ULONGEST arg0)
 {
   gdb_printf (gdb_stdlog, "-> %s->goto_record (...)\n", this->beneath ()->shortname ());
   this->beneath ()->goto_record (arg0);
-  gdb_printf (gdb_stdlog, "<- %s->goto_record (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ULONGEST (arg0), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->goto_record (%s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ULONGEST (arg0).c_str ());
 }
 
 void
@@ -4223,11 +4196,11 @@  debug_target::insn_history (int arg0, gdb_disassembly_flags arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->insn_history (...)\n", this->beneath ()->shortname ());
   this->beneath ()->insn_history (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->insn_history (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_gdb_disassembly_flags (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->insn_history (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str (),
+	      target_debug_print_gdb_disassembly_flags (arg1).c_str ());
 }
 
 void
@@ -4247,13 +4220,12 @@  debug_target::insn_history_from (ULONGEST arg0, int arg1, gdb_disassembly_flags
 {
   gdb_printf (gdb_stdlog, "-> %s->insn_history_from (...)\n", this->beneath ()->shortname ());
   this->beneath ()->insn_history_from (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->insn_history_from (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ULONGEST (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_gdb_disassembly_flags (arg2), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->insn_history_from (%s, %s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ULONGEST (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str (),
+	      target_debug_print_gdb_disassembly_flags (arg2).c_str ());
 }
 
 void
@@ -4273,13 +4245,12 @@  debug_target::insn_history_range (ULONGEST arg0, ULONGEST arg1, gdb_disassembly_
 {
   gdb_printf (gdb_stdlog, "-> %s->insn_history_range (...)\n", this->beneath ()->shortname ());
   this->beneath ()->insn_history_range (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->insn_history_range (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ULONGEST (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ULONGEST (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_gdb_disassembly_flags (arg2), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->insn_history_range (%s, %s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ULONGEST (arg0).c_str (),
+	      target_debug_print_ULONGEST (arg1).c_str (),
+	      target_debug_print_gdb_disassembly_flags (arg2).c_str ());
 }
 
 void
@@ -4299,11 +4270,11 @@  debug_target::call_history (int arg0, record_print_flags arg1)
 {
   gdb_printf (gdb_stdlog, "-> %s->call_history (...)\n", this->beneath ()->shortname ());
   this->beneath ()->call_history (arg0, arg1);
-  gdb_printf (gdb_stdlog, "<- %s->call_history (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_int (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_record_print_flags (arg1), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->call_history (%s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_int (arg0).c_str (),
+	      target_debug_print_record_print_flags (arg1).c_str ());
 }
 
 void
@@ -4323,13 +4294,12 @@  debug_target::call_history_from (ULONGEST arg0, int arg1, record_print_flags arg
 {
   gdb_printf (gdb_stdlog, "-> %s->call_history_from (...)\n", this->beneath ()->shortname ());
   this->beneath ()->call_history_from (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->call_history_from (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ULONGEST (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_record_print_flags (arg2), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->call_history_from (%s, %s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ULONGEST (arg0).c_str (),
+	      target_debug_print_int (arg1).c_str (),
+	      target_debug_print_record_print_flags (arg2).c_str ());
 }
 
 void
@@ -4349,13 +4319,12 @@  debug_target::call_history_range (ULONGEST arg0, ULONGEST arg1, record_print_fla
 {
   gdb_printf (gdb_stdlog, "-> %s->call_history_range (...)\n", this->beneath ()->shortname ());
   this->beneath ()->call_history_range (arg0, arg1, arg2);
-  gdb_printf (gdb_stdlog, "<- %s->call_history_range (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_ULONGEST (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_ULONGEST (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_record_print_flags (arg2), gdb_stdlog);
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->call_history_range (%s, %s, %s)\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_ULONGEST (arg0).c_str (),
+	      target_debug_print_ULONGEST (arg1).c_str (),
+	      target_debug_print_record_print_flags (arg2).c_str ());
 }
 
 bool
@@ -4376,10 +4345,10 @@  debug_target::augmented_libraries_svr4_read ()
   gdb_printf (gdb_stdlog, "-> %s->augmented_libraries_svr4_read (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->augmented_libraries_svr4_read ();
-  gdb_printf (gdb_stdlog, "<- %s->augmented_libraries_svr4_read (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->augmented_libraries_svr4_read () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -4401,10 +4370,10 @@  debug_target::get_unwinder ()
   gdb_printf (gdb_stdlog, "-> %s->get_unwinder (...)\n", this->beneath ()->shortname ());
   const struct frame_unwind * result
     = this->beneath ()->get_unwinder ();
-  gdb_printf (gdb_stdlog, "<- %s->get_unwinder (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_const_frame_unwind_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_unwinder () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_frame_unwind_p (result).c_str ());
   return result;
 }
 
@@ -4426,10 +4395,10 @@  debug_target::get_tailcall_unwinder ()
   gdb_printf (gdb_stdlog, "-> %s->get_tailcall_unwinder (...)\n", this->beneath ()->shortname ());
   const struct frame_unwind * result
     = this->beneath ()->get_tailcall_unwinder ();
-  gdb_printf (gdb_stdlog, "<- %s->get_tailcall_unwinder (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_const_frame_unwind_p (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->get_tailcall_unwinder () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_const_frame_unwind_p (result).c_str ());
   return result;
 }
 
@@ -4449,8 +4418,9 @@  debug_target::prepare_to_generate_core ()
 {
   gdb_printf (gdb_stdlog, "-> %s->prepare_to_generate_core (...)\n", this->beneath ()->shortname ());
   this->beneath ()->prepare_to_generate_core ();
-  gdb_printf (gdb_stdlog, "<- %s->prepare_to_generate_core (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->prepare_to_generate_core ()\n",
+	      this->beneath ()->shortname ());
 }
 
 void
@@ -4469,8 +4439,9 @@  debug_target::done_generating_core ()
 {
   gdb_printf (gdb_stdlog, "-> %s->done_generating_core (...)\n", this->beneath ()->shortname ());
   this->beneath ()->done_generating_core ();
-  gdb_printf (gdb_stdlog, "<- %s->done_generating_core (", this->beneath ()->shortname ());
-  gdb_puts (")\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->done_generating_core ()\n",
+	      this->beneath ()->shortname ());
 }
 
 bool
@@ -4491,10 +4462,10 @@  debug_target::supports_memory_tagging ()
   gdb_printf (gdb_stdlog, "-> %s->supports_memory_tagging (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->supports_memory_tagging ();
-  gdb_printf (gdb_stdlog, "<- %s->supports_memory_tagging (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->supports_memory_tagging () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -4516,17 +4487,14 @@  debug_target::fetch_memtags (CORE_ADDR arg0, size_t arg1, gdb::byte_vector &arg2
   gdb_printf (gdb_stdlog, "-> %s->fetch_memtags (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->fetch_memtags (arg0, arg1, arg2, arg3);
-  gdb_printf (gdb_stdlog, "<- %s->fetch_memtags (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_size_t (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_gdb_byte_vector_r (arg2), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg3), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->fetch_memtags (%s, %s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_size_t (arg1).c_str (),
+	      target_debug_print_gdb_byte_vector_r (arg2).c_str (),
+	      target_debug_print_int (arg3).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -4548,17 +4516,14 @@  debug_target::store_memtags (CORE_ADDR arg0, size_t arg1, const gdb::byte_vector
   gdb_printf (gdb_stdlog, "-> %s->store_memtags (...)\n", this->beneath ()->shortname ());
   bool result
     = this->beneath ()->store_memtags (arg0, arg1, arg2, arg3);
-  gdb_printf (gdb_stdlog, "<- %s->store_memtags (", this->beneath ()->shortname ());
-  gdb_puts (target_debug_print_CORE_ADDR (arg0), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_size_t (arg1), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_const_gdb_byte_vector_r (arg2), gdb_stdlog);
-  gdb_puts (", ", gdb_stdlog);
-  gdb_puts (target_debug_print_int (arg3), gdb_stdlog);
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_bool (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->store_memtags (%s, %s, %s, %s) = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_CORE_ADDR (arg0).c_str (),
+	      target_debug_print_size_t (arg1).c_str (),
+	      target_debug_print_const_gdb_byte_vector_r (arg2).c_str (),
+	      target_debug_print_int (arg3).c_str (),
+	      target_debug_print_bool (result).c_str ());
   return result;
 }
 
@@ -4580,9 +4545,9 @@  debug_target::fetch_x86_xsave_layout ()
   gdb_printf (gdb_stdlog, "-> %s->fetch_x86_xsave_layout (...)\n", this->beneath ()->shortname ());
   x86_xsave_layout result
     = this->beneath ()->fetch_x86_xsave_layout ();
-  gdb_printf (gdb_stdlog, "<- %s->fetch_x86_xsave_layout (", this->beneath ()->shortname ());
-  gdb_puts (") = ", gdb_stdlog);
-  target_debug_print_x86_xsave_layout (result);
-  gdb_puts ("\n", gdb_stdlog);
+  gdb_printf (gdb_stdlog,
+	      "<- %s->fetch_x86_xsave_layout () = %s\n",
+	      this->beneath ()->shortname (),
+	      target_debug_print_x86_xsave_layout (result).c_str ());
   return result;
 }