Style the "Reading symbols" message
authorTom Tromey <tom@tromey.com>
Thu, 6 Sep 2018 20:49:39 +0000 (14:49 -0600)
committerTom Tromey <tom@tromey.com>
Fri, 28 Dec 2018 19:49:52 +0000 (12:49 -0700)
The "Reading symbols" message does not use ui-out (perhaps it
should?), so this styles it using the low-level API.

gdb/ChangeLog
2018-12-28  Tom Tromey  <tom@tromey.com>

* symfile.c (symbol_file_add_with_addrs): Style file name.

gdb/testsuite/ChangeLog
2018-12-28  Tom Tromey  <tom@tromey.com>

* gdb.base/style.exp: Add test for styling of "Reading symbols"
message.

gdb/ChangeLog
gdb/symfile.c
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.base/style.exp

index dd19b80..1067973 100644 (file)
@@ -1,5 +1,9 @@
 2018-12-28  Tom Tromey  <tom@tromey.com>
 
+       * symfile.c (symbol_file_add_with_addrs): Style file name.
+
+2018-12-28  Tom Tromey  <tom@tromey.com>
+
        * top.c (print_gdb_version): Style gdb version number.
 
 2018-12-28  Tom Tromey  <tom@tromey.com>
index 991b71f..ce1cc9e 100644 (file)
@@ -58,6 +58,7 @@
 #include "cli/cli-utils.h"
 #include "common/byte-vector.h"
 #include "selftest.h"
+#include "cli/cli-style.h"
 
 #include <sys/types.h>
 #include <fcntl.h>
@@ -1110,7 +1111,11 @@ symbol_file_add_with_addrs (bfd *abfd, const char *name,
       if (deprecated_pre_add_symbol_hook)
        deprecated_pre_add_symbol_hook (name);
       else
-       printf_filtered (_("Reading symbols from %s...\n"), name);
+       {
+         puts_filtered (_("Reading symbols from "));
+         fputs_styled (name, file_name_style.style (), gdb_stdout);
+         puts_filtered ("...\n");
+       }
     }
   syms_from_objfile (objfile, addrs, add_flags);
 
index 3d262d3..ceef606 100644 (file)
@@ -1,5 +1,10 @@
 2018-12-28  Tom Tromey  <tom@tromey.com>
 
+       * gdb.base/style.exp: Add test for styling of "Reading symbols"
+       message.
+
+2018-12-28  Tom Tromey  <tom@tromey.com>
+
        * gdb.base/style.exp: Add test for version number styling.
 
 2018-12-28  Tom Tromey  <tom@tromey.com>
index b72c2ab..8bababc 100644 (file)
@@ -50,4 +50,8 @@ save_vars { env(TERM) } {
 
     gdb_test "" "\033\\\[35;1mGNU gdb.*\033\\\[m.*" \
        "version is styled"
+
+    set quoted [string_to_regexp $binfile]
+    gdb_test "file $binfile" \
+       "Reading symbols from \033\\\[32m${quoted}\033\\\[m..." \
 }