Remove a cleanup from remote.c

This removes a cleanup from remote.c by using std::string to construct
the qSupported packet.

Tested by the buildbot.

ChangeLog
2018-05-04  Tom Tromey  <tom@tromey.com>

	* remote.c (remote_query_supported_append): Change type.
	(remote_check_symbols): Update.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index c20f2a9d0b53dd9740748c76fe773c2783f58d93..b7e372fe215541ed201e12bf2a4599f13295cfe8 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2018-05-04  Tom Tromey  <tom@tromey.com>
+
+       * remote.c (remote_query_supported_append): Change type.
+       (remote_check_symbols): Update.
+
 2018-05-04  Paul Pluzhnikov  <ppluzhnikov@google.com>
 
        PR gdb/11420

diff --git a/gdb/remote.c b/gdb/remote.c
index 89cba546673fe9de37f45d6d1be5d0632dccfad9..66099cfe16af924ebebd0dbb9ee88953565b4d3d 100644 (file)
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -4957,13 +4957,12 @@ register_remote_support_xml (const char *xml)
 #endif
 }
 
-static char *
-remote_query_supported_append (char *msg, const char *append)
+static void
+remote_query_supported_append (std::string *msg, const char *append)
 {
-  if (msg)
-    return reconcat (msg, msg, ";", append, (char *) NULL);
-  else
-    return xstrdup (append);
+  if (!msg->empty ())
+    msg->append (";");
+  msg->append (append);
 }
 
 static void
@@ -4984,48 +4983,45 @@ remote_query_supported (void)
   rs->buf[0] = 0;
   if (packet_support (PACKET_qSupported) != PACKET_DISABLE)
     {
-      char *q = NULL;
-      struct cleanup *old_chain = make_cleanup (free_current_contents, &q);
+      std::string q;
 
       if (packet_set_cmd_state (PACKET_multiprocess_feature) != AUTO_BOOLEAN_FALSE)
-       q = remote_query_supported_append (q, "multiprocess+");
+       remote_query_supported_append (&q, "multiprocess+");
 
       if (packet_set_cmd_state (PACKET_swbreak_feature) != AUTO_BOOLEAN_FALSE)
-       q = remote_query_supported_append (q, "swbreak+");
+       remote_query_supported_append (&q, "swbreak+");
       if (packet_set_cmd_state (PACKET_hwbreak_feature) != AUTO_BOOLEAN_FALSE)
-       q = remote_query_supported_append (q, "hwbreak+");
+       remote_query_supported_append (&q, "hwbreak+");
 
-      q = remote_query_supported_append (q, "qRelocInsn+");
+      remote_query_supported_append (&q, "qRelocInsn+");
 
       if (packet_set_cmd_state (PACKET_fork_event_feature)
          != AUTO_BOOLEAN_FALSE)
-       q = remote_query_supported_append (q, "fork-events+");
+       remote_query_supported_append (&q, "fork-events+");
       if (packet_set_cmd_state (PACKET_vfork_event_feature)
          != AUTO_BOOLEAN_FALSE)
-       q = remote_query_supported_append (q, "vfork-events+");
+       remote_query_supported_append (&q, "vfork-events+");
       if (packet_set_cmd_state (PACKET_exec_event_feature)
          != AUTO_BOOLEAN_FALSE)
-       q = remote_query_supported_append (q, "exec-events+");
+       remote_query_supported_append (&q, "exec-events+");
 
       if (packet_set_cmd_state (PACKET_vContSupported) != AUTO_BOOLEAN_FALSE)
-       q = remote_query_supported_append (q, "vContSupported+");
+       remote_query_supported_append (&q, "vContSupported+");
 
       if (packet_set_cmd_state (PACKET_QThreadEvents) != AUTO_BOOLEAN_FALSE)
-       q = remote_query_supported_append (q, "QThreadEvents+");
+       remote_query_supported_append (&q, "QThreadEvents+");
 
       if (packet_set_cmd_state (PACKET_no_resumed) != AUTO_BOOLEAN_FALSE)
-       q = remote_query_supported_append (q, "no-resumed+");
+       remote_query_supported_append (&q, "no-resumed+");
 
       /* Keep this one last to work around a gdbserver <= 7.10 bug in
         the qSupported:xmlRegisters=i386 handling.  */
       if (remote_support_xml != NULL
          && packet_support (PACKET_qXfer_features) != PACKET_DISABLE)
-       q = remote_query_supported_append (q, remote_support_xml);
-
-      q = reconcat (q, "qSupported:", q, (char *) NULL);
-      putpkt (q);
+       remote_query_supported_append (&q, remote_support_xml);
 
-      do_cleanups (old_chain);
+      q = "qSupported:" + q;
+      putpkt (q.c_str ());
 
       getpkt (&rs->buf, &rs->buf_size, 0);