gdbsupport: change xml_escape_text_append's parameter from pointer to reference

The passed in string can't be nullptr, it makes more sense to pass in a
reference.

Change-Id: Idc8bd38abe1d6d9b44aa227d7856956848c233b3

diff --git a/gdb/unittests/xml-utils-selftests.c b/gdb/unittests/xml-utils-selftests.c
index 08a48f686aa9b864019385ef9d8bfde0db381e15..f86e1e1bc077ac89e810ccb242ad2d56978aab07 100644 (file)
--- a/gdb/unittests/xml-utils-selftests.c
+++ b/gdb/unittests/xml-utils-selftests.c
@@ -40,7 +40,7 @@ static void test_xml_escape_text_append ()
   const char *input = "<this isn't=\"xml\"> &";
   const char *expected_output
     = "foo<xml>&lt;this isn&apos;t=&quot;xml&quot;&gt; &amp;";
-  xml_escape_text_append (&actual_output, input);
+  xml_escape_text_append (actual_output, input);
 
   SELF_CHECK (actual_output == expected_output);
 }

diff --git a/gdbserver/linux-low.cc b/gdbserver/linux-low.cc
index 6f96e16d6f0404de03ab4711e1c25cb4d8b4a134..5e412315e6dd2631e41ce00f28ed29cba2e0ca96 100644 (file)
--- a/gdbserver/linux-low.cc
+++ b/gdbserver/linux-low.cc
@@ -6520,7 +6520,7 @@ read_link_map (std::string &document, CORE_ADDR lmid, CORE_ADDR lm_addr,
       if (libname[0] != '\0')
        {
          string_appendf (document, "<library name=\"");
-         xml_escape_text_append (&document, (char *) libname);
+         xml_escape_text_append (document, (char *) libname);
          string_appendf (document, "\" lm=\"0x%s\" l_addr=\"0x%s\" "
                          "l_ld=\"0x%s\" lmid=\"0x%s\"/>",
                          paddress (lm_addr), paddress (l_addr),

diff --git a/gdbserver/netbsd-low.cc b/gdbserver/netbsd-low.cc
index f05bcd4e173fb7522b106fd90c2b717bfcccb3a9..af2c6c82b908d57ccc371132471b24640f1128ca 100644 (file)
--- a/gdbserver/netbsd-low.cc
+++ b/gdbserver/netbsd-low.cc
@@ -1086,7 +1086,7 @@ netbsd_qxfer_libraries_svr4 (const pid_t pid, const char *annex,
                }
 
              string_appendf (document, "<library name=\"");
-             xml_escape_text_append (&document, (char *) libname);
+             xml_escape_text_append (document, (char *) libname);
              string_appendf (document, "\" lm=\"0x%lx\" "
                              "l_addr=\"0x%lx\" l_ld=\"0x%lx\"/>",
                              (unsigned long) lm_addr, (unsigned long) l_addr,

diff --git a/gdbsupport/xml-utils.cc b/gdbsupport/xml-utils.cc
index ec5e943828b78f837d427f0493d23c555223c1b5..e47e23ced590e5644912b80b2c6ef7be6f224e2a 100644 (file)
--- a/gdbsupport/xml-utils.cc
+++ b/gdbsupport/xml-utils.cc
@@ -27,7 +27,7 @@ xml_escape_text (const char *text)
 {
   std::string result;
 
-  xml_escape_text_append (&result, text);
+  xml_escape_text_append (result, text);
 
   return result;
 }
@@ -35,29 +35,29 @@ xml_escape_text (const char *text)
 /* See xml-utils.h.  */
 
 void
-xml_escape_text_append (std::string *result, const char *text)
+xml_escape_text_append (std::string &result, const char *text)
 {
   /* Expand the result.  */
   for (int i = 0; text[i] != '\0'; i++)
     switch (text[i])
       {
       case '\'':
-       *result += "'";
+       result += "'";
        break;
       case '\"':
-       *result += """;
+       result += """;
        break;
       case '&':
-       *result += "&";
+       result += "&";
        break;
       case '<':
-       *result += "&lt;";
+       result += "&lt;";
        break;
       case '>':
-       *result += "&gt;";
+       result += "&gt;";
        break;
       default:
-       *result += text[i];
+       result += text[i];
        break;
       }
 }

diff --git a/gdbsupport/xml-utils.h b/gdbsupport/xml-utils.h
index 4df2f8ab6f62192ea86213bfcea9390b2c7b7313..695263c5b379b15d3ef23040b30cc66614ac3536 100644 (file)
--- a/gdbsupport/xml-utils.h
+++ b/gdbsupport/xml-utils.h
@@ -28,6 +28,6 @@ extern std::string xml_escape_text (const char *text);
 /* Append TEXT to RESULT, with special characters replaced by entity
    references.  */
 
-extern void xml_escape_text_append (std::string *result, const char *text);
+extern void xml_escape_text_append (std::string &result, const char *text);
 
 #endif /* COMMON_XML_UTILS_H */