aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Tromey <tom@tromey.com>2017-05-04 16:01:41 -0600
committerTom Tromey <tom@tromey.com>2017-09-29 21:12:16 -0600
commitdcd5ddccd7d7917caf42b8d429a7535857c6e9b2 (patch)
tree1e0745cd1b9ddf545721edf4eb1a7aa5abe596c7
parent45d288cc64e08dd68938f1d9bcc70906b5343652 (diff)
downloadgdb-dcd5ddccd7d7917caf42b8d429a7535857c6e9b2.zip
gdb-dcd5ddccd7d7917caf42b8d429a7535857c6e9b2.tar.gz
gdb-dcd5ddccd7d7917caf42b8d429a7535857c6e9b2.tar.bz2
Use std::string in mi-main.c
Change a couple of spots in mi-main.c to use std::string, and change one place to use field_fmt. This removes some cleanups. gdb/ChangeLog 2017-09-29 Tom Tromey <tom@tromey.com> * mi/mi-main.c (mi_execute_cli_command): Use std::string. (mi_execute_async_cli_command): Likewise. (mi_cmd_trace_frame_collected): Use field_fmt.
-rw-r--r--gdb/ChangeLog6
-rw-r--r--gdb/mi/mi-main.c41
2 files changed, 16 insertions, 31 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index df58772..230735b 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,11 @@
2017-09-29 Tom Tromey <tom@tromey.com>
+ * mi/mi-main.c (mi_execute_cli_command): Use std::string.
+ (mi_execute_async_cli_command): Likewise.
+ (mi_cmd_trace_frame_collected): Use field_fmt.
+
+2017-09-29 Tom Tromey <tom@tromey.com>
+
* mi/mi-main.c (mi_cmd_data_write_memory_bytes): Use
gdb::byte_vector.
diff --git a/gdb/mi/mi-main.c b/gdb/mi/mi-main.c
index e1ba8e2..2d560a4 100644
--- a/gdb/mi/mi-main.c
+++ b/gdb/mi/mi-main.c
@@ -2269,41 +2269,29 @@ mi_execute_cli_command (const char *cmd, int args_p, const char *args)
{
if (cmd != 0)
{
- struct cleanup *old_cleanups;
- char *run;
+ std::string run = cmd;
if (args_p)
- run = xstrprintf ("%s %s", cmd, args);
- else
- run = xstrdup (cmd);
+ run = run + " " + args;
if (mi_debug_p)
/* FIXME: gdb_???? */
fprintf_unfiltered (gdb_stdout, "cli=%s run=%s\n",
- cmd, run);
- old_cleanups = make_cleanup (xfree, run);
- execute_command (run, 0 /* from_tty */ );
- do_cleanups (old_cleanups);
- return;
+ cmd, run.c_str ());
+ execute_command (&run[0], 0 /* from_tty */ );
}
}
void
mi_execute_async_cli_command (const char *cli_command, char **argv, int argc)
{
- struct cleanup *old_cleanups;
- char *run;
+ std::string run = cli_command;
+ if (argc)
+ run = run + " " + *argv;
if (mi_async_p ())
- run = xstrprintf ("%s %s&", cli_command, argc ? *argv : "");
- else
- run = xstrprintf ("%s %s", cli_command, argc ? *argv : "");
- old_cleanups = make_cleanup (xfree, run);
+ run += "&";
- execute_command (run, 0 /* from_tty */ );
-
- /* Do this before doing any printing. It would appear that some
- print code leaves garbage around in the buffer. */
- do_cleanups (old_cleanups);
+ execute_command (&run[0], 0 /* from_tty */ );
}
void
@@ -2806,14 +2794,10 @@ mi_cmd_trace_frame_collected (const char *command, char **argv, int argc)
{
struct cleanup *cleanups;
int tvar;
- char *tsvname;
int i;
ui_out_emit_list list_emitter (uiout, "tvars");
- tsvname = NULL;
- cleanups = make_cleanup (free_current_contents, &tsvname);
-
for (i = 0; VEC_iterate (int, tinfo->tvars, i, tvar); i++)
{
struct trace_state_variable *tsv;
@@ -2824,10 +2808,7 @@ mi_cmd_trace_frame_collected (const char *command, char **argv, int argc)
if (tsv != NULL)
{
- tsvname = (char *) xrealloc (tsvname, strlen (tsv->name) + 2);
- tsvname[0] = '$';
- strcpy (tsvname + 1, tsv->name);
- uiout->field_string ("name", tsvname);
+ uiout->field_fmt ("name", "$%s", tsv->name);
tsv->value_known = target_get_trace_state_variable_value (tsv->number,
&tsv->value);
@@ -2839,8 +2820,6 @@ mi_cmd_trace_frame_collected (const char *command, char **argv, int argc)
uiout->field_skip ("current");
}
}
-
- do_cleanups (cleanups);
}
/* Memory. */