Fix pager bugs with style output
[deliverable/binutils-gdb.git] / gdb / printcmd.c
index 696285ed8a42839d4ca477a06d99fa6a241e22e8..03763d577ab2dfd7b15f8ad45f81ba50eb43543f 100644 (file)
@@ -1,6 +1,6 @@
 /* Print values for GNU debugger GDB.
 
-   Copyright (C) 1986-2018 Free Software Foundation, Inc.
+   Copyright (C) 1986-2019 Free Software Foundation, Inc.
 
    This file is part of GDB.
 
@@ -46,7 +46,8 @@
 #include "arch-utils.h"
 #include "cli/cli-utils.h"
 #include "cli/cli-script.h"
-#include "format.h"
+#include "cli/cli-style.h"
+#include "common/format.h"
 #include "source.h"
 #include "common/byte-vector.h"
 
@@ -534,7 +535,7 @@ print_address_symbolic (struct gdbarch *gdbarch, CORE_ADDR addr,
     fputs_filtered ("<*", stream);
   else
     fputs_filtered ("<", stream);
-  fputs_filtered (name.c_str (), stream);
+  fputs_styled (name.c_str (), function_name_style.style (), stream);
   if (offset != 0)
     fprintf_filtered (stream, "+%u", (unsigned int) offset);
 
@@ -542,10 +543,10 @@ print_address_symbolic (struct gdbarch *gdbarch, CORE_ADDR addr,
      line # of this addr, if we have it; else line # of the nearest symbol.  */
   if (print_symbol_filename && !filename.empty ())
     {
+      fputs_filtered (line == -1 ? " in " : " at ", stream);
+      fputs_styled (filename.c_str (), file_name_style.style (), stream);
       if (line != -1)
-       fprintf_filtered (stream, " at %s:%d", filename.c_str (), line);
-      else
-       fprintf_filtered (stream, " in %s", filename.c_str ());
+       fprintf_filtered (stream, ":%d", line);
     }
   if (unmapped)
     fputs_filtered ("*>", stream);
@@ -609,7 +610,7 @@ build_address_symbolic (struct gdbarch *gdbarch,
         pointer is <function+3>.  This matches the ISA behavior.  */
       addr = gdbarch_addr_bits_remove (gdbarch, addr);
 
-      name_location = BLOCK_START (SYMBOL_BLOCK_VALUE (symbol));
+      name_location = BLOCK_ENTRY_PC (SYMBOL_BLOCK_VALUE (symbol));
       if (do_demangle || asm_demangle)
        name_temp = SYMBOL_PRINT_NAME (symbol);
       else
@@ -691,7 +692,7 @@ void
 print_address (struct gdbarch *gdbarch,
               CORE_ADDR addr, struct ui_file *stream)
 {
-  fputs_filtered (paddress (gdbarch, addr), stream);
+  fputs_styled (paddress (gdbarch, addr), address_style.style (), stream);
   print_address_symbolic (gdbarch, addr, stream, asm_demangle, " ");
 }
 
@@ -724,7 +725,7 @@ print_address_demangle (const struct value_print_options *opts,
 {
   if (opts->addressprint)
     {
-      fputs_filtered (paddress (gdbarch, addr), stream);
+      fputs_styled (paddress (gdbarch, addr), address_style.style (), stream);
       print_address_symbolic (gdbarch, addr, stream, do_demangle, " ");
     }
   else
@@ -1271,7 +1272,6 @@ static void
 info_symbol_command (const char *arg, int from_tty)
 {
   struct minimal_symbol *msymbol;
-  struct objfile *objfile;
   struct obj_section *osect;
   CORE_ADDR addr, sect_addr;
   int matches = 0;
@@ -1281,79 +1281,81 @@ info_symbol_command (const char *arg, int from_tty)
     error_no_arg (_("address"));
 
   addr = parse_and_eval_address (arg);
-  ALL_OBJSECTIONS (objfile, osect)
-  {
-    /* Only process each object file once, even if there's a separate
-       debug file.  */
-    if (objfile->separate_debug_objfile_backlink)
-      continue;
-
-    sect_addr = overlay_mapped_address (addr, osect);
-
-    if (obj_section_addr (osect) <= sect_addr
-       && sect_addr < obj_section_endaddr (osect)
-       && (msymbol
-           = lookup_minimal_symbol_by_pc_section (sect_addr, osect).minsym))
+  for (objfile *objfile : current_program_space->objfiles ())
+    ALL_OBJFILE_OSECTIONS (objfile, osect)
       {
-       const char *obj_name, *mapped, *sec_name, *msym_name;
-       const char *loc_string;
-       struct cleanup *old_chain;
-
-       matches = 1;
-       offset = sect_addr - MSYMBOL_VALUE_ADDRESS (objfile, msymbol);
-       mapped = section_is_mapped (osect) ? _("mapped") : _("unmapped");
-       sec_name = osect->the_bfd_section->name;
-       msym_name = MSYMBOL_PRINT_NAME (msymbol);
-
-       /* Don't print the offset if it is zero.
-          We assume there's no need to handle i18n of "sym + offset".  */
-       std::string string_holder;
-       if (offset)
+       /* Only process each object file once, even if there's a separate
+          debug file.  */
+       if (objfile->separate_debug_objfile_backlink)
+         continue;
+
+       sect_addr = overlay_mapped_address (addr, osect);
+
+       if (obj_section_addr (osect) <= sect_addr
+           && sect_addr < obj_section_endaddr (osect)
+           && (msymbol
+               = lookup_minimal_symbol_by_pc_section (sect_addr,
+                                                      osect).minsym))
          {
-           string_holder = string_printf ("%s + %u", msym_name, offset);
-           loc_string = string_holder.c_str ();
-         }
-       else
-         loc_string = msym_name;
-
-       gdb_assert (osect->objfile && objfile_name (osect->objfile));
-       obj_name = objfile_name (osect->objfile);
-
-       if (MULTI_OBJFILE_P ())
-         if (pc_in_unmapped_range (addr, osect))
-           if (section_is_overlay (osect))
-             printf_filtered (_("%s in load address range of "
-                                "%s overlay section %s of %s\n"),
-                              loc_string, mapped, sec_name, obj_name);
-           else
-             printf_filtered (_("%s in load address range of "
-                                "section %s of %s\n"),
-                              loc_string, sec_name, obj_name);
-         else
-           if (section_is_overlay (osect))
-             printf_filtered (_("%s in %s overlay section %s of %s\n"),
-                              loc_string, mapped, sec_name, obj_name);
-           else
-             printf_filtered (_("%s in section %s of %s\n"),
-                              loc_string, sec_name, obj_name);
-       else
-         if (pc_in_unmapped_range (addr, osect))
-           if (section_is_overlay (osect))
-             printf_filtered (_("%s in load address range of %s overlay "
-                                "section %s\n"),
-                              loc_string, mapped, sec_name);
+           const char *obj_name, *mapped, *sec_name, *msym_name;
+           const char *loc_string;
+
+           matches = 1;
+           offset = sect_addr - MSYMBOL_VALUE_ADDRESS (objfile, msymbol);
+           mapped = section_is_mapped (osect) ? _("mapped") : _("unmapped");
+           sec_name = osect->the_bfd_section->name;
+           msym_name = MSYMBOL_PRINT_NAME (msymbol);
+
+           /* Don't print the offset if it is zero.
+              We assume there's no need to handle i18n of "sym + offset".  */
+           std::string string_holder;
+           if (offset)
+             {
+               string_holder = string_printf ("%s + %u", msym_name, offset);
+               loc_string = string_holder.c_str ();
+             }
            else
-             printf_filtered (_("%s in load address range of section %s\n"),
-                              loc_string, sec_name);
-         else
-           if (section_is_overlay (osect))
-             printf_filtered (_("%s in %s overlay section %s\n"),
-                              loc_string, mapped, sec_name);
+             loc_string = msym_name;
+
+           gdb_assert (osect->objfile && objfile_name (osect->objfile));
+           obj_name = objfile_name (osect->objfile);
+
+           if (MULTI_OBJFILE_P ())
+             if (pc_in_unmapped_range (addr, osect))
+               if (section_is_overlay (osect))
+                 printf_filtered (_("%s in load address range of "
+                                    "%s overlay section %s of %s\n"),
+                                  loc_string, mapped, sec_name, obj_name);
+               else
+                 printf_filtered (_("%s in load address range of "
+                                    "section %s of %s\n"),
+                                  loc_string, sec_name, obj_name);
+             else
+               if (section_is_overlay (osect))
+                 printf_filtered (_("%s in %s overlay section %s of %s\n"),
+                                  loc_string, mapped, sec_name, obj_name);
+               else
+                 printf_filtered (_("%s in section %s of %s\n"),
+                                  loc_string, sec_name, obj_name);
            else
-             printf_filtered (_("%s in section %s\n"),
-                              loc_string, sec_name);
+             if (pc_in_unmapped_range (addr, osect))
+               if (section_is_overlay (osect))
+                 printf_filtered (_("%s in load address range of %s overlay "
+                                    "section %s\n"),
+                                  loc_string, mapped, sec_name);
+               else
+                 printf_filtered
+                   (_("%s in load address range of section %s\n"),
+                    loc_string, sec_name);
+             else
+               if (section_is_overlay (osect))
+                 printf_filtered (_("%s in %s overlay section %s\n"),
+                                  loc_string, mapped, sec_name);
+               else
+                 printf_filtered (_("%s in section %s\n"),
+                                  loc_string, sec_name);
+         }
       }
-  }
   if (matches == 0)
     printf_filtered (_("No symbol matches %s.\n"), arg);
 }
@@ -1403,14 +1405,17 @@ info_address_command (const char *exp, int from_tty)
          fprintf_symbol_filtered (gdb_stdout, exp,
                                   current_language->la_language, DMGL_ANSI);
          printf_filtered ("\" is at ");
-         fputs_filtered (paddress (gdbarch, load_addr), gdb_stdout);
+         fputs_styled (paddress (gdbarch, load_addr), address_style.style (),
+                       gdb_stdout);
          printf_filtered (" in a file compiled without debugging");
          section = MSYMBOL_OBJ_SECTION (objfile, msymbol.minsym);
          if (section_is_overlay (section))
            {
              load_addr = overlay_unmapped_address (load_addr, section);
              printf_filtered (",\n -- loaded at ");
-             fputs_filtered (paddress (gdbarch, load_addr), gdb_stdout);
+             fputs_styled (paddress (gdbarch, load_addr),
+                           address_style.style (),
+                           gdb_stdout);
              printf_filtered (" in overlay section %s",
                               section->the_bfd_section->name);
            }
@@ -1450,12 +1455,14 @@ info_address_command (const char *exp, int from_tty)
     case LOC_LABEL:
       printf_filtered ("a label at address ");
       load_addr = SYMBOL_VALUE_ADDRESS (sym);
-      fputs_filtered (paddress (gdbarch, load_addr), gdb_stdout);
+      fputs_styled (paddress (gdbarch, load_addr), address_style.style (),
+                   gdb_stdout);
       if (section_is_overlay (section))
        {
          load_addr = overlay_unmapped_address (load_addr, section);
          printf_filtered (",\n -- loaded at ");
-         fputs_filtered (paddress (gdbarch, load_addr), gdb_stdout);
+         fputs_styled (paddress (gdbarch, load_addr), address_style.style (),
+                       gdb_stdout);
          printf_filtered (" in overlay section %s",
                           section->the_bfd_section->name);
        }
@@ -1484,12 +1491,14 @@ info_address_command (const char *exp, int from_tty)
     case LOC_STATIC:
       printf_filtered (_("static storage at address "));
       load_addr = SYMBOL_VALUE_ADDRESS (sym);
-      fputs_filtered (paddress (gdbarch, load_addr), gdb_stdout);
+      fputs_styled (paddress (gdbarch, load_addr), address_style.style (),
+                   gdb_stdout);
       if (section_is_overlay (section))
        {
          load_addr = overlay_unmapped_address (load_addr, section);
          printf_filtered (_(",\n -- loaded at "));
-         fputs_filtered (paddress (gdbarch, load_addr), gdb_stdout);
+         fputs_styled (paddress (gdbarch, load_addr), address_style.style (),
+                       gdb_stdout);
          printf_filtered (_(" in overlay section %s"),
                           section->the_bfd_section->name);
        }
@@ -1520,13 +1529,15 @@ info_address_command (const char *exp, int from_tty)
 
     case LOC_BLOCK:
       printf_filtered (_("a function at address "));
-      load_addr = BLOCK_START (SYMBOL_BLOCK_VALUE (sym));
-      fputs_filtered (paddress (gdbarch, load_addr), gdb_stdout);
+      load_addr = BLOCK_ENTRY_PC (SYMBOL_BLOCK_VALUE (sym));
+      fputs_styled (paddress (gdbarch, load_addr), address_style.style (),
+                   gdb_stdout);
       if (section_is_overlay (section))
        {
          load_addr = overlay_unmapped_address (load_addr, section);
          printf_filtered (_(",\n -- loaded at "));
-         fputs_filtered (paddress (gdbarch, load_addr), gdb_stdout);
+         fputs_styled (paddress (gdbarch, load_addr), address_style.style (),
+                       gdb_stdout);
          printf_filtered (_(" in overlay section %s"),
                           section->the_bfd_section->name);
        }
@@ -1556,12 +1567,15 @@ info_address_command (const char *exp, int from_tty)
              {
                load_addr = BMSYMBOL_VALUE_ADDRESS (msym);
                printf_filtered (_("static storage at address "));
-               fputs_filtered (paddress (gdbarch, load_addr), gdb_stdout);
+               fputs_styled (paddress (gdbarch, load_addr),
+                             address_style.style (), gdb_stdout);
                if (section_is_overlay (section))
                  {
                    load_addr = overlay_unmapped_address (load_addr, section);
                    printf_filtered (_(",\n -- loaded at "));
-                   fputs_filtered (paddress (gdbarch, load_addr), gdb_stdout);
+                   fputs_styled (paddress (gdbarch, load_addr),
+                                 address_style.style (),
+                                 gdb_stdout);
                    printf_filtered (_(" in overlay section %s"),
                                     section->the_bfd_section->name);
                  }
@@ -2157,7 +2171,10 @@ print_variable_and_value (const char *name, struct symbol *var,
   if (!name)
     name = SYMBOL_PRINT_NAME (var);
 
-  fprintf_filtered (stream, "%s%s = ", n_spaces (2 * indent), name);
+  fputs_filtered (n_spaces (2 * indent), stream);
+  fputs_styled (name, variable_name_style.style (), stream);
+  fputs_filtered (" = ", stream);
+
   TRY
     {
       struct value *val;
@@ -2201,7 +2218,10 @@ printf_c_string (struct ui_file *stream, const char *format,
   tem = value_as_address (value);
   if (tem == 0)
     {
+      DIAGNOSTIC_PUSH
+      DIAGNOSTIC_IGNORE_FORMAT_NONLITERAL
       fprintf_filtered (stream, format, "(null)");
+      DIAGNOSTIC_POP
       return;
     }
 
@@ -2222,7 +2242,10 @@ printf_c_string (struct ui_file *stream, const char *format,
     read_memory (tem, str, j);
   str[j] = 0;
 
+  DIAGNOSTIC_PUSH
+  DIAGNOSTIC_IGNORE_FORMAT_NONLITERAL
   fprintf_filtered (stream, format, (char *) str);
+  DIAGNOSTIC_POP
 }
 
 /* Subroutine of ui_printf to simplify it.
@@ -2246,7 +2269,10 @@ printf_wide_c_string (struct ui_file *stream, const char *format,
   tem = value_as_address (value);
   if (tem == 0)
     {
+      DIAGNOSTIC_PUSH
+      DIAGNOSTIC_IGNORE_FORMAT_NONLITERAL
       fprintf_filtered (stream, format, "(null)");
+      DIAGNOSTIC_POP
       return;
     }
 
@@ -2273,7 +2299,10 @@ printf_wide_c_string (struct ui_file *stream, const char *format,
                             &output, translit_char);
   obstack_grow_str0 (&output, "");
 
+  DIAGNOSTIC_PUSH
+  DIAGNOSTIC_IGNORE_FORMAT_NONLITERAL
   fprintf_filtered (stream, format, obstack_base (&output));
+  DIAGNOSTIC_POP
 }
 
 /* Subroutine of ui_printf to simplify it.
@@ -2286,7 +2315,6 @@ printf_floating (struct ui_file *stream, const char *format,
   /* Parameter data.  */
   struct type *param_type = value_type (value);
   struct gdbarch *gdbarch = get_type_arch (param_type);
-  enum bfd_endian byte_order = gdbarch_byte_order (gdbarch);
 
   /* Determine target type corresponding to the format string.  */
   struct type *fmt_type;
@@ -2402,13 +2430,19 @@ printf_pointer (struct ui_file *stream, const char *format,
       *fmt_p++ = 'l';
       *fmt_p++ = 'x';
       *fmt_p++ = '\0';
+      DIAGNOSTIC_PUSH
+      DIAGNOSTIC_IGNORE_FORMAT_NONLITERAL
       fprintf_filtered (stream, fmt, val);
+      DIAGNOSTIC_POP
     }
   else
     {
       *fmt_p++ = 's';
       *fmt_p++ = '\0';
+      DIAGNOSTIC_PUSH
+      DIAGNOSTIC_IGNORE_FORMAT_NONLITERAL
       fprintf_filtered (stream, fmt, "(nil)");
+      DIAGNOSTIC_POP
     }
 }
 
@@ -2509,8 +2543,11 @@ ui_printf (const char *arg, struct ui_file *stream)
                                         &output, translit_char);
              obstack_grow_str0 (&output, "");
 
+             DIAGNOSTIC_PUSH
+             DIAGNOSTIC_IGNORE_FORMAT_NONLITERAL
              fprintf_filtered (stream, current_substring,
                                 obstack_base (&output));
+             DIAGNOSTIC_POP
            }
            break;
          case long_long_arg:
@@ -2518,7 +2555,10 @@ ui_printf (const char *arg, struct ui_file *stream)
            {
              long long val = value_as_long (val_args[i]);
 
+             DIAGNOSTIC_PUSH
+             DIAGNOSTIC_IGNORE_FORMAT_NONLITERAL
               fprintf_filtered (stream, current_substring, val);
+             DIAGNOSTIC_POP
              break;
            }
 #else
@@ -2528,14 +2568,20 @@ ui_printf (const char *arg, struct ui_file *stream)
            {
              int val = value_as_long (val_args[i]);
 
+             DIAGNOSTIC_PUSH
+             DIAGNOSTIC_IGNORE_FORMAT_NONLITERAL
               fprintf_filtered (stream, current_substring, val);
+             DIAGNOSTIC_POP
              break;
            }
          case long_arg:
            {
              long val = value_as_long (val_args[i]);
 
+             DIAGNOSTIC_PUSH
+             DIAGNOSTIC_IGNORE_FORMAT_NONLITERAL
               fprintf_filtered (stream, current_substring, val);
+             DIAGNOSTIC_POP
              break;
            }
          /* Handles floating-point values.  */
@@ -2559,7 +2605,10 @@ ui_printf (const char *arg, struct ui_file *stream)
               have modified GCC to include -Wformat-security by
               default, which will warn here if there is no
               argument.  */
+           DIAGNOSTIC_PUSH
+           DIAGNOSTIC_IGNORE_FORMAT_NONLITERAL
            fprintf_filtered (stream, current_substring, 0);
+           DIAGNOSTIC_POP
            break;
          default:
            internal_error (__FILE__, __LINE__,
@@ -2578,6 +2627,8 @@ static void
 printf_command (const char *arg, int from_tty)
 {
   ui_printf (arg, gdb_stdout);
+  reset_terminal_style (gdb_stdout);
+  wrap_here ("");
   gdb_flush (gdb_stdout);
 }
 
This page took 0.035859 seconds and 4 git commands to generate.