/* MI Command Set - varobj commands.
- Copyright (C) 2000-2017 Free Software Foundation, Inc.
+ Copyright (C) 2000-2023 Free Software Foundation, Inc.
Contributed by Cygnus Solutions (a Red Hat company).
#include "mi-getopt.h"
#include "gdbthread.h"
#include "mi-parse.h"
-
-extern unsigned int varobjdebug; /* defined in varobj.c. */
+#include "gdbsupport/gdb_optional.h"
+#include "inferior.h"
static void varobj_update_one (struct varobj *var,
enum print_values print_values,
- int is_explicit);
+ bool is_explicit);
static int mi_print_value_p (struct varobj *var,
enum print_values print_values);
{
std::string exp = varobj_get_expression (var);
- uiout->field_string ("exp", exp.c_str ());
+ uiout->field_string ("exp", exp);
}
- uiout->field_int ("numchild", varobj_get_num_children (var));
+ uiout->field_signed ("numchild", varobj_get_num_children (var));
if (mi_print_value_p (var, print_values))
{
std::string val = varobj_get_value (var);
- uiout->field_string ("value", val.c_str ());
+ uiout->field_string ("value", val);
}
std::string type = varobj_get_type (var);
if (!type.empty ())
- uiout->field_string ("type", type.c_str ());
+ uiout->field_string ("type", type);
thread_id = varobj_get_thread_id (var);
if (thread_id > 0)
- uiout->field_int ("thread-id", thread_id);
+ uiout->field_signed ("thread-id", thread_id);
if (varobj_get_frozen (var))
- uiout->field_int ("frozen", 1);
+ uiout->field_signed ("frozen", 1);
gdb::unique_xmalloc_ptr<char> display_hint = varobj_get_display_hint (var);
if (display_hint)
uiout->field_string ("displayhint", display_hint.get ());
if (varobj_is_dynamic_p (var))
- uiout->field_int ("dynamic", 1);
+ uiout->field_signed ("dynamic", 1);
}
/* VAROBJ operations */
void
-mi_cmd_var_create (const char *command, char **argv, int argc)
+mi_cmd_var_create (const char *command, const char *const *argv, int argc)
{
struct ui_out *uiout = current_uiout;
CORE_ADDR frameaddr = 0;
struct varobj *var;
- char *name;
- char *frame;
- char *expr;
- struct cleanup *old_cleanups;
+ const char *frame;
+ const char *expr;
enum varobj_type var_type;
if (argc != 3)
error (_("-var-create: Usage: NAME FRAME EXPRESSION."));
- name = xstrdup (argv[0]);
- /* Add cleanup for name. Must be free_current_contents as name can
- be reallocated. */
- old_cleanups = make_cleanup (free_current_contents, &name);
-
- frame = xstrdup (argv[1]);
- make_cleanup (xfree, frame);
-
- expr = xstrdup (argv[2]);
- make_cleanup (xfree, expr);
+ frame = argv[1];
+ expr = argv[2];
+ const char *name = argv[0];
+ std::string gen_name;
if (strcmp (name, "-") == 0)
{
- xfree (name);
- name = varobj_gen_name ();
+ gen_name = varobj_gen_name ();
+ name = gen_name.c_str ();
}
- else if (!isalpha (*name))
+ else if (!isalpha (name[0]))
error (_("-var-create: name of object must begin with a letter"));
if (strcmp (frame, "*") == 0)
}
if (varobjdebug)
- fprintf_unfiltered (gdb_stdlog,
- "Name=\"%s\", Frame=\"%s\" (%s), Expression=\"%s\"\n",
- name, frame, hex_string (frameaddr), expr);
+ gdb_printf (gdb_stdlog,
+ "Name=\"%s\", Frame=\"%s\" (%s), Expression=\"%s\"\n",
+ name, frame, hex_string (frameaddr), expr);
var = varobj_create (name, expr, frameaddr, var_type);
print_varobj (var, PRINT_ALL_VALUES, 0 /* don't print expression */);
- uiout->field_int ("has_more", varobj_has_more (var, 0));
-
- do_cleanups (old_cleanups);
+ uiout->field_signed ("has_more", varobj_has_more (var, 0));
}
void
-mi_cmd_var_delete (const char *command, char **argv, int argc)
+mi_cmd_var_delete (const char *command, const char *const *argv, int argc)
{
- char *name;
+ const char *name;
struct varobj *var;
int numdel;
int children_only_p = 0;
- struct cleanup *old_cleanups;
struct ui_out *uiout = current_uiout;
if (argc < 1 || argc > 2)
error (_("-var-delete: Usage: [-c] EXPRESSION."));
- name = xstrdup (argv[0]);
- /* Add cleanup for name. Must be free_current_contents as name can
- be reallocated. */
- old_cleanups = make_cleanup (free_current_contents, &name);
+ name = argv[0];
/* If we have one single argument it cannot be '-c' or any string
starting with '-'. */
if (strcmp (name, "-c") != 0)
error (_("-var-delete: Invalid option."));
children_only_p = 1;
- do_cleanups (old_cleanups);
- name = xstrdup (argv[1]);
- old_cleanups = make_cleanup (free_current_contents, &name);
+ name = argv[1];
}
/* If we didn't error out, now NAME contains the name of the
numdel = varobj_delete (var, children_only_p);
- uiout->field_int ("ndeleted", numdel);
-
- do_cleanups (old_cleanups);
+ uiout->field_signed ("ndeleted", numdel);
}
/* Parse a string argument into a format value. */
}
void
-mi_cmd_var_set_format (const char *command, char **argv, int argc)
+mi_cmd_var_set_format (const char *command, const char *const *argv, int argc)
{
enum varobj_display_formats format;
struct varobj *var;
/* Report the value in the new format. */
std::string val = varobj_get_value (var);
- uiout->field_string ("value", val.c_str ());
+ uiout->field_string ("value", val);
}
void
-mi_cmd_var_set_visualizer (const char *command, char **argv, int argc)
+mi_cmd_var_set_visualizer (const char *command, const char *const *argv,
+ int argc)
{
struct varobj *var;
if (argc != 2)
- error (_("Usage: NAME VISUALIZER_FUNCTION."));
+ error (_("-var-set-visualizer: Usage: NAME VISUALIZER_FUNCTION."));
var = varobj_get_handle (argv[0]);
}
void
-mi_cmd_var_set_frozen (const char *command, char **argv, int argc)
+mi_cmd_var_set_frozen (const char *command, const char *const *argv, int argc)
{
struct varobj *var;
- int frozen;
+ bool frozen;
if (argc != 2)
- error (_("-var-set-format: Usage: NAME FROZEN_FLAG."));
+ error (_("-var-set-frozen: Usage: NAME FROZEN_FLAG."));
var = varobj_get_handle (argv[0]);
if (strcmp (argv[1], "0") == 0)
- frozen = 0;
+ frozen = false;
else if (strcmp (argv[1], "1") == 0)
- frozen = 1;
+ frozen = true;
else
error (_("Invalid flag value"));
}
void
-mi_cmd_var_show_format (const char *command, char **argv, int argc)
+mi_cmd_var_show_format (const char *command, const char *const *argv, int argc)
{
struct ui_out *uiout = current_uiout;
enum varobj_display_formats format;
}
void
-mi_cmd_var_info_num_children (const char *command, char **argv, int argc)
+mi_cmd_var_info_num_children (const char *command, const char *const *argv,
+ int argc)
{
struct ui_out *uiout = current_uiout;
struct varobj *var;
/* Get varobj handle, if a valid var obj name was specified. */
var = varobj_get_handle (argv[0]);
- uiout->field_int ("numchild", varobj_get_num_children (var));
+ uiout->field_signed ("numchild", varobj_get_num_children (var));
}
/* Return 1 if given the argument PRINT_VALUES we should display
if (type == NULL)
return 1;
else
- {
- type = check_typedef (type);
+ return mi_simple_type_p (type);
+}
+
+/* See mi-cmds.h. */
- /* For PRINT_SIMPLE_VALUES, only print the value if it has a type
- and that type is not a compound type. */
- return (TYPE_CODE (type) != TYPE_CODE_ARRAY
- && TYPE_CODE (type) != TYPE_CODE_STRUCT
- && TYPE_CODE (type) != TYPE_CODE_UNION);
+bool
+mi_simple_type_p (struct type *type)
+{
+ type = check_typedef (type);
+
+ if (TYPE_IS_REFERENCE (type))
+ type = check_typedef (type->target_type ());
+
+ switch (type->code ())
+ {
+ case TYPE_CODE_ARRAY:
+ case TYPE_CODE_STRUCT:
+ case TYPE_CODE_UNION:
+ return false;
+ default:
+ return true;
}
}
void
-mi_cmd_var_list_children (const char *command, char **argv, int argc)
+mi_cmd_var_list_children (const char *command, const char *const *argv,
+ int argc)
{
struct ui_out *uiout = current_uiout;
struct varobj *var;
- VEC(varobj_p) *children;
- struct varobj *child;
enum print_values print_values;
- int ix;
int from, to;
if (argc < 1 || argc > 4)
to = -1;
}
- children = varobj_list_children (var, &from, &to);
- uiout->field_int ("numchild", to - from);
+ const std::vector<varobj *> &children
+ = varobj_list_children (var, &from, &to);
+
+ uiout->field_signed ("numchild", to - from);
if (argc == 2 || argc == 4)
print_values = mi_parse_print_values (argv[0]);
else
if (from < to)
{
- struct cleanup *cleanup_children;
-
- if (mi_version (uiout) == 1)
- cleanup_children
- = make_cleanup_ui_out_tuple_begin_end (uiout, "children");
- else
- cleanup_children
- = make_cleanup_ui_out_list_begin_end (uiout, "children");
- for (ix = from;
- ix < to && VEC_iterate (varobj_p, children, ix, child);
- ++ix)
+ ui_out_emit_list list_emitter (uiout, "children");
+ for (int ix = from; ix < to && ix < children.size (); ix++)
{
- struct cleanup *cleanup_child;
+ ui_out_emit_tuple child_emitter (uiout, "child");
- cleanup_child = make_cleanup_ui_out_tuple_begin_end (uiout, "child");
- print_varobj (child, print_values, 1 /* print expression */);
- do_cleanups (cleanup_child);
+ print_varobj (children[ix], print_values, 1 /* print expression */);
}
- do_cleanups (cleanup_children);
}
- uiout->field_int ("has_more", varobj_has_more (var, to));
+ uiout->field_signed ("has_more", varobj_has_more (var, to));
}
void
-mi_cmd_var_info_type (const char *command, char **argv, int argc)
+mi_cmd_var_info_type (const char *command, const char *const *argv, int argc)
{
struct ui_out *uiout = current_uiout;
struct varobj *var;
var = varobj_get_handle (argv[0]);
std::string type_name = varobj_get_type (var);
- uiout->field_string ("type", type_name.c_str ());
+ uiout->field_string ("type", type_name);
}
void
-mi_cmd_var_info_path_expression (const char *command, char **argv, int argc)
+mi_cmd_var_info_path_expression (const char *command, const char *const *argv,
+ int argc)
{
struct ui_out *uiout = current_uiout;
struct varobj *var;
if (argc != 1)
- error (_("Usage: NAME."));
+ error (_("-var-info-path-expression: Usage: NAME."));
/* Get varobj handle, if a valid var obj name was specified. */
var = varobj_get_handle (argv[0]);
}
void
-mi_cmd_var_info_expression (const char *command, char **argv, int argc)
+mi_cmd_var_info_expression (const char *command, const char *const *argv,
+ int argc)
{
struct ui_out *uiout = current_uiout;
const struct language_defn *lang;
lang = varobj_get_language (var);
- uiout->field_string ("lang", lang->la_natural_name);
+ uiout->field_string ("lang", lang->natural_name ());
std::string exp = varobj_get_expression (var);
- uiout->field_string ("exp", exp.c_str ());
+ uiout->field_string ("exp", exp);
}
void
-mi_cmd_var_show_attributes (const char *command, char **argv, int argc)
+mi_cmd_var_show_attributes (const char *command, const char *const *argv,
+ int argc)
{
struct ui_out *uiout = current_uiout;
int attr;
}
void
-mi_cmd_var_evaluate_expression (const char *command, char **argv, int argc)
+mi_cmd_var_evaluate_expression (const char *command, const char *const *argv,
+ int argc)
{
struct ui_out *uiout = current_uiout;
struct varobj *var;
enum varobj_display_formats format;
int formatFound;
int oind;
- char *oarg;
+ const char *oarg;
enum opt
{
{
std::string val = varobj_get_formatted_value (var, format);
- uiout->field_string ("value", val.c_str ());
+ uiout->field_string ("value", val);
}
else
{
std::string val = varobj_get_value (var);
- uiout->field_string ("value", val.c_str ());
+ uiout->field_string ("value", val);
}
}
void
-mi_cmd_var_assign (const char *command, char **argv, int argc)
+mi_cmd_var_assign (const char *command, const char *const *argv, int argc)
{
struct ui_out *uiout = current_uiout;
struct varobj *var;
"expression to variable object"));
std::string val = varobj_get_value (var);
- uiout->field_string ("value", val.c_str ());
+ uiout->field_string ("value", val);
}
-/* Type used for parameters passing to mi_cmd_var_update_iter. */
-
-struct mi_cmd_var_update
- {
- int only_floating;
- enum print_values print_values;
- };
-
/* Helper for mi_cmd_var_update - update each VAR. */
static void
-mi_cmd_var_update_iter (struct varobj *var, void *data_pointer)
+mi_cmd_var_update_iter (struct varobj *var, bool only_floating,
+ enum print_values print_values)
{
- struct mi_cmd_var_update *data = (struct mi_cmd_var_update *) data_pointer;
- int thread_id, thread_stopped;
+ bool thread_stopped;
- thread_id = varobj_get_thread_id (var);
+ int thread_id = varobj_get_thread_id (var);
- if (thread_id == -1
- && (ptid_equal (inferior_ptid, null_ptid)
- || is_stopped (inferior_ptid)))
- thread_stopped = 1;
+ if (thread_id == -1)
+ {
+ thread_stopped = (inferior_ptid == null_ptid
+ || inferior_thread ()->state == THREAD_STOPPED);
+ }
else
{
- struct thread_info *tp = find_thread_global_id (thread_id);
+ thread_info *tp = find_thread_global_id (thread_id);
- if (tp)
- thread_stopped = is_stopped (tp->ptid);
- else
- thread_stopped = 1;
+ thread_stopped = (tp == NULL
+ || tp->state == THREAD_STOPPED);
}
if (thread_stopped
- && (!data->only_floating || varobj_floating_p (var)))
- varobj_update_one (var, data->print_values, 0 /* implicit */);
+ && (!only_floating || varobj_floating_p (var)))
+ varobj_update_one (var, print_values, false /* implicit */);
}
void
-mi_cmd_var_update (const char *command, char **argv, int argc)
+mi_cmd_var_update (const char *command, const char *const *argv, int argc)
{
struct ui_out *uiout = current_uiout;
- struct cleanup *cleanup;
- char *name;
+ const char *name;
enum print_values print_values;
if (argc != 1 && argc != 2)
else
print_values = PRINT_NO_VALUES;
- if (mi_version (uiout) <= 1)
- cleanup = make_cleanup_ui_out_tuple_begin_end (uiout, "changelist");
- else
- cleanup = make_cleanup_ui_out_list_begin_end (uiout, "changelist");
+ ui_out_emit_list list_emitter (uiout, "changelist");
/* Check if the parameter is a "*", which means that we want to
update all variables. */
if ((*name == '*' || *name == '@') && (*(name + 1) == '\0'))
{
- struct mi_cmd_var_update data;
-
- data.only_floating = (*name == '@');
- data.print_values = print_values;
-
/* varobj_update_one automatically updates all the children of
VAROBJ. Therefore update each VAROBJ only once by iterating
only the root VAROBJs. */
- all_root_varobjs (mi_cmd_var_update_iter, &data);
+ all_root_varobjs ([=] (varobj *var)
+ { mi_cmd_var_update_iter (var, *name == '0', print_values); });
}
else
{
/* Get varobj handle, if a valid var obj name was specified. */
struct varobj *var = varobj_get_handle (name);
- varobj_update_one (var, print_values, 1 /* explicit */);
+ varobj_update_one (var, print_values, true /* explicit */);
}
-
- do_cleanups (cleanup);
}
/* Helper for mi_cmd_var_update(). */
static void
varobj_update_one (struct varobj *var, enum print_values print_values,
- int is_explicit)
+ bool is_explicit)
{
struct ui_out *uiout = current_uiout;
- VEC (varobj_update_result) *changes;
- varobj_update_result *r;
- int i;
-
- changes = varobj_update (&var, is_explicit);
+
+ std::vector<varobj_update_result> changes = varobj_update (&var, is_explicit);
- for (i = 0; VEC_iterate (varobj_update_result, changes, i, r); ++i)
+ for (const varobj_update_result &r : changes)
{
int from, to;
- struct cleanup *cleanup = make_cleanup (null_cleanup, NULL);
- if (mi_version (uiout) > 1)
- make_cleanup_ui_out_tuple_begin_end (uiout, NULL);
- uiout->field_string ("name", varobj_get_objname (r->varobj));
+ ui_out_emit_tuple tuple_emitter (uiout, nullptr);
+ uiout->field_string ("name", varobj_get_objname (r.varobj));
- switch (r->status)
+ switch (r.status)
{
case VAROBJ_IN_SCOPE:
- if (mi_print_value_p (r->varobj, print_values))
+ if (mi_print_value_p (r.varobj, print_values))
{
- std::string val = varobj_get_value (r->varobj);
+ std::string val = varobj_get_value (r.varobj);
- uiout->field_string ("value", val.c_str ());
+ uiout->field_string ("value", val);
}
uiout->field_string ("in_scope", "true");
break;
- case VAROBJ_NOT_IN_SCOPE:
- uiout->field_string ("in_scope", "false");
+ case VAROBJ_NOT_IN_SCOPE:
+ uiout->field_string ("in_scope", "false");
+ break;
+ case VAROBJ_INVALID:
+ uiout->field_string ("in_scope", "invalid");
break;
- case VAROBJ_INVALID:
- uiout->field_string ("in_scope", "invalid");
- break;
}
- if (r->status != VAROBJ_INVALID)
+ if (r.status != VAROBJ_INVALID)
{
- if (r->type_changed)
+ if (r.type_changed)
uiout->field_string ("type_changed", "true");
else
uiout->field_string ("type_changed", "false");
}
- if (r->type_changed)
+ if (r.type_changed)
{
- std::string type_name = varobj_get_type (r->varobj);
+ std::string type_name = varobj_get_type (r.varobj);
- uiout->field_string ("new_type", type_name.c_str ());
+ uiout->field_string ("new_type", type_name);
}
- if (r->type_changed || r->children_changed)
- uiout->field_int ("new_num_children",
- varobj_get_num_children (r->varobj));
+ if (r.type_changed || r.children_changed)
+ uiout->field_signed ("new_num_children",
+ varobj_get_num_children (r.varobj));
gdb::unique_xmalloc_ptr<char> display_hint
- = varobj_get_display_hint (r->varobj);
+ = varobj_get_display_hint (r.varobj);
if (display_hint)
uiout->field_string ("displayhint", display_hint.get ());
- if (varobj_is_dynamic_p (r->varobj))
- uiout->field_int ("dynamic", 1);
+ if (varobj_is_dynamic_p (r.varobj))
+ uiout->field_signed ("dynamic", 1);
- varobj_get_child_range (r->varobj, &from, &to);
- uiout->field_int ("has_more", varobj_has_more (r->varobj, to));
+ varobj_get_child_range (r.varobj, &from, &to);
+ uiout->field_signed ("has_more", varobj_has_more (r.varobj, to));
- if (r->newobj)
+ if (!r.newobj.empty ())
{
- int j;
- varobj_p child;
- struct cleanup *cleanup;
+ ui_out_emit_list list_emitter (uiout, "new_children");
- cleanup = make_cleanup_ui_out_list_begin_end (uiout, "new_children");
- for (j = 0; VEC_iterate (varobj_p, r->newobj, j, child); ++j)
+ for (varobj *child : r.newobj)
{
- struct cleanup *cleanup_child;
-
- cleanup_child
- = make_cleanup_ui_out_tuple_begin_end (uiout, NULL);
+ ui_out_emit_tuple inner_tuple_emitter (uiout, NULL);
print_varobj (child, print_values, 1 /* print_expression */);
- do_cleanups (cleanup_child);
}
-
- do_cleanups (cleanup);
- VEC_free (varobj_p, r->newobj);
- r->newobj = NULL; /* Paranoia. */
}
-
- do_cleanups (cleanup);
}
- VEC_free (varobj_update_result, changes);
}
void
-mi_cmd_enable_pretty_printing (const char *command, char **argv, int argc)
+mi_cmd_enable_pretty_printing (const char *command, const char *const *argv,
+ int argc)
{
if (argc != 0)
error (_("-enable-pretty-printing: no arguments allowed"));
}
void
-mi_cmd_var_set_update_range (const char *command, char **argv, int argc)
+mi_cmd_var_set_update_range (const char *command, const char *const *argv,
+ int argc)
{
struct varobj *var;
int from, to;
projects / binutils-gdb.git / blobdiff