Use filtered output in *-tdep commands
authorTom Tromey <tom@tromey.com>
Sat, 25 Dec 2021 05:15:42 +0000 (22:15 -0700)
committerTom Tromey <tom@tromey.com>
Wed, 5 Jan 2022 18:36:33 +0000 (11:36 -0700)
Various targets introduce their own commands, which then use
unfiltered output.  It's better to use filtered output by default, so
this patch fixes the instances I found.

gdb/avr-tdep.c
gdb/hppa-tdep.c
gdb/i386-tdep.c
gdb/mips-tdep.c
gdb/s12z-tdep.c

index 9ead25d5386ba46516b5c6d29c04762d64b54b73..7f79ef94bdf4be34c591a6beedeb7a1db243a6a9 100644 (file)
@@ -1589,7 +1589,7 @@ avr_io_reg_read_command (const char *args, int from_tty)
       return;
     }
 
-  printf_unfiltered (_("Target has %u io registers:\n\n"), nreg);
+  printf_filtered (_("Target has %u io registers:\n\n"), nreg);
 
   /* only fetch up to 8 registers at a time to keep the buffer small */
   int step = 8;
index 6c6e538e8228797cf069c07ff40fd238806fba49..32c543570458cbaf5a5ff844f4b01def4cd8b22b 100644 (file)
@@ -2572,19 +2572,19 @@ unwind_command (const char *exp, int from_tty)
 
   if (!u)
     {
-      printf_unfiltered ("Can't find unwind table entry for %s\n", exp);
+      printf_filtered ("Can't find unwind table entry for %s\n", exp);
       return;
     }
 
-  printf_unfiltered ("unwind_table_entry (%s):\n", host_address_to_string (u));
+  printf_filtered ("unwind_table_entry (%s):\n", host_address_to_string (u));
 
-  printf_unfiltered ("\tregion_start = %s\n", hex_string (u->region_start));
+  printf_filtered ("\tregion_start = %s\n", hex_string (u->region_start));
 
-  printf_unfiltered ("\tregion_end = %s\n", hex_string (u->region_end));
+  printf_filtered ("\tregion_end = %s\n", hex_string (u->region_end));
 
-#define pif(FLD) if (u->FLD) printf_unfiltered (" "#FLD);
+#define pif(FLD) if (u->FLD) printf_filtered (" "#FLD);
 
-  printf_unfiltered ("\n\tflags =");
+  printf_filtered ("\n\tflags =");
   pif (Cannot_unwind);
   pif (Millicode);
   pif (Millicode_save_sr0);
@@ -2609,9 +2609,9 @@ unwind_command (const char *exp, int from_tty)
   pif (Large_frame);
   pif (alloca_frame);
 
-  putchar_unfiltered ('\n');
+  putchar_filtered ('\n');
 
-#define pin(FLD) printf_unfiltered ("\t"#FLD" = 0x%x\n", u->FLD);
+#define pin(FLD) printf_filtered ("\t"#FLD" = 0x%x\n", u->FLD);
 
   pin (Region_description);
   pin (Entry_FR);
@@ -2620,26 +2620,26 @@ unwind_command (const char *exp, int from_tty)
 
   if (u->stub_unwind.stub_type)
     {
-      printf_unfiltered ("\tstub type = ");
+      printf_filtered ("\tstub type = ");
       switch (u->stub_unwind.stub_type)
        {
          case LONG_BRANCH:
-           printf_unfiltered ("long branch\n");
+           printf_filtered ("long branch\n");
            break;
          case PARAMETER_RELOCATION:
-           printf_unfiltered ("parameter relocation\n");
+           printf_filtered ("parameter relocation\n");
            break;
          case EXPORT:
-           printf_unfiltered ("export\n");
+           printf_filtered ("export\n");
            break;
          case IMPORT:
-           printf_unfiltered ("import\n");
+           printf_filtered ("import\n");
            break;
          case IMPORT_SHLIB:
-           printf_unfiltered ("import shlib\n");
+           printf_filtered ("import shlib\n");
            break;
          default:
-           printf_unfiltered ("unknown (%d)\n", u->stub_unwind.stub_type);
+           printf_filtered ("unknown (%d)\n", u->stub_unwind.stub_type);
        }
     }
 }
index 6d8a43fb4f8c9f8fa2d8faad497c6a1fae45391a..aaa5592f40a9c00b8a9e0a5a59d718e36ae94a5e 100644 (file)
@@ -8992,14 +8992,14 @@ i386_mpx_info_bounds (const char *args, int from_tty)
   if (gdbarch_bfd_arch_info (gdbarch)->arch != bfd_arch_i386
       || !i386_mpx_enabled ())
     {
-      printf_unfiltered (_("Intel Memory Protection Extensions not "
-                          "supported on this target.\n"));
+      printf_filtered (_("Intel Memory Protection Extensions not "
+                        "supported on this target.\n"));
       return;
     }
 
   if (args == NULL)
     {
-      printf_unfiltered (_("Address of pointer variable expected.\n"));
+      printf_filtered (_("Address of pointer variable expected.\n"));
       return;
     }
 
index 7f8018360dd80bc48db6abd8e15e6b2dd8e80337..e5f8c6b20533c8950a5bd36d34cfe7f34fa602da 100644 (file)
@@ -6916,7 +6916,7 @@ show_mipsfpu_command (const char *args, int from_tty)
 
   if (gdbarch_bfd_arch_info (target_gdbarch ())->arch != bfd_arch_mips)
     {
-      printf_unfiltered
+      printf_filtered
        ("The MIPS floating-point coprocessor is unknown "
         "because the current architecture is not MIPS.\n");
       return;
@@ -6937,11 +6937,11 @@ show_mipsfpu_command (const char *args, int from_tty)
       internal_error (__FILE__, __LINE__, _("bad switch"));
     }
   if (mips_fpu_type_auto)
-    printf_unfiltered ("The MIPS floating-point coprocessor "
+    printf_filtered ("The MIPS floating-point coprocessor "
                       "is set automatically (currently %s)\n",
                       fpu);
   else
-    printf_unfiltered
+    printf_filtered
       ("The MIPS floating-point coprocessor is assumed to be %s\n", fpu);
 }
 
index 24ebc6a8516f5818403a331ce8bb5919987b65c4..3f9740f0dfeac708f7e2a4cdee78fa76820968e6 100644 (file)
@@ -635,7 +635,7 @@ show_bdccsr_command (const char *args, int from_tty)
   struct string_file output;
   target_rcmd ("bdccsr", &output);
 
-  printf_unfiltered ("The current BDCCSR value is %s\n", output.string().c_str());
+  printf_filtered ("The current BDCCSR value is %s\n", output.string().c_str());
 }
 
 static struct gdbarch *