From: Marco Barisione <mbarisione@undo.io>
Date: Wed, 19 May 2021 15:48:05 +0000 (+0100)
Subject: gdb: Move definitions of std::string overloads in ui_out to the header
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=0d7e3cd15fa770e172f442e1a984809fa8ac0fdc;p=binutils-gdb.git

gdb: Move definitions of std::string overloads in ui_out to the header

These methods are just trivial wrappers around the versions accepting
a char pointer.  By moving them to the header the compiler can inline
them.

gdb/ChangeLog:

	* ui-out.c (ui_out::field_string): Move to ui-out.h.
	(ui_out::text): Ditto.
	* ui-out.h (class ui_out): Add definitions of
	ui_out::field_string and ui_out::text which were previously
	defined in ui-out.c.
---

diff --git a/gdb/ui-out.c b/gdb/ui-out.c
index c091af6cad3..982d157a932 100644
--- a/gdb/ui-out.c
+++ b/gdb/ui-out.c
@@ -527,13 +527,6 @@ ui_out::field_string (const char *fldname, const char *string,
   do_field_string (fldno, width, align, fldname, string, style);
 }
 
-void
-ui_out::field_string (const char *fldname, const std::string &string,
-		      const ui_file_style &style)
-{
-  field_string (fldname, string.c_str (), style);
-}
-
 /* VARARGS */
 void
 ui_out::field_fmt (const char *fldname, const char *format, ...)
@@ -582,12 +575,6 @@ ui_out::text (const char *string)
   do_text (string);
 }
 
-void
-ui_out::text (const std::string &string)
-{
-  text (string.c_str ());
-}
-
 void
 ui_out::call_do_message (const ui_file_style &style, const char *format,
 			 ...)
diff --git a/gdb/ui-out.h b/gdb/ui-out.h
index 46dfe28c1e1..ecf62e8b601 100644
--- a/gdb/ui-out.h
+++ b/gdb/ui-out.h
@@ -192,7 +192,10 @@ class ui_out
   void field_string (const char *fldname, const char *string,
 		     const ui_file_style &style = ui_file_style ());
   void field_string (const char *fldname, const std::string &string,
-		     const ui_file_style &style = ui_file_style ());
+		     const ui_file_style &style = ui_file_style ())
+  {
+    field_string (fldname, string.c_str (), style);
+  }
   void field_stream (const char *fldname, string_file &stream,
 		     const ui_file_style &style = ui_file_style ());
   void field_skip (const char *fldname);
@@ -204,7 +207,7 @@ class ui_out
 
   void spaces (int numspaces);
   void text (const char *string);
-  void text (const std::string &string);
+  void text (const std::string &string) { text (string.c_str ()); }
 
   /* Output a printf-style formatted string.  In addition to the usual
      printf format specs, this supports a few GDB-specific