aboutsummaryrefslogtreecommitdiff
path: root/gdb/c-varobj.c
diff options
context:
space:
mode:
authorSimon Marchi <simon.marchi@ericsson.com>2017-11-23 11:00:56 -0500
committerSimon Marchi <simon.marchi@ericsson.com>2017-11-23 11:00:56 -0500
commit4c37490d92c0168e5192ce109a791c4b3fec9f8e (patch)
tree51ca155671475582d88cc8cb52f8d4259a4b5813 /gdb/c-varobj.c
parenteb02c04dc3600e18bb06dc1ef8fc3cf9a967d550 (diff)
downloadgdb-4c37490d92c0168e5192ce109a791c4b3fec9f8e.zip
gdb-4c37490d92c0168e5192ce109a791c4b3fec9f8e.tar.gz
gdb-4c37490d92c0168e5192ce109a791c4b3fec9f8e.tar.bz2
Change int -> bool where applicable throughout varobj
This patch changes all the "int" I could find in varobj.{c,h} that are really boolean values to use bool. I followed the ramifications (parameters and return values of exported functions), so the changes spilled a bit on other, related files (ada-varobj.c and c-varobj.c). gdb/ChangeLog: * ada-varobj.c (ada_value_is_changeable_p): Change int to bool where applicable. (ada_value_has_mutated): Likewise. * c-varobj.c (varobj_is_anonymous_child): Likewise. (c_is_path_expr_parent): Likewise. * mi/mi-cmd-var.c (varobj_update_one): Likewise. (mi_cmd_var_set_frozen): Likewise. (mi_cmd_var_update_iter): Likewise. (mi_cmd_var_update): Likewise. * varobj.c (pretty_printing): Likewise. (varobj_enable_pretty_printing): Likewise. (struct varobj_root) <floating, is_valid>: Likewise. (struct varobj_dynamic) <children_requested>: Likewise. (delete_variable): Likewise. (delete_variable_1): Likewise. (install_variable): Likewise. (update_type_if_necessary): Likewise. (install_new_value): Likewise. (value_of_root): Likewise. (is_root_p): Likewise. (varobj_create): Likewise. (varobj_delete): Likewise. (varobj_has_more): Likewise. (varobj_set_frozen): Likewise. (varobj_get_frozen): Likewise. (install_dynamic_child): Likewise. (dynamic_varobj_has_child_method): Likewise. (update_dynamic_varobj_children): Likewise. (varobj_get_num_children): Likewise. (varobj_list_children): Likewise. (is_path_expr_parent): Likewise. (varobj_default_is_path_expr_parent): Likewise. (varobj_is_dynamic_p): Likewise. (varobj_set_value): Likewise. (varobj_value_has_mutated): Likewise. (varobj_update): Likewise. (check_scope): Likewise. (value_of_root_1): Likewise. (varobj_value_get_print_value): Likewise. (varobj_editable_p): Likewise. (varobj_value_is_changeable_p): Likewise. (varobj_floating_p): Likewise. (varobj_default_value_is_changeable_p): Likewise. (varobj_invalidate_iter): Likewise. * varobj.h (struct varobj_update_result) <type_changed, children_changed, changed, value_installed>: Likewise. (struct varobj) <updated, frozen, not_fetched>: Likewise. (struct lang_varobj_ops) <value_is_changeable_p, value_has_mutated, is_path_expr_parent>: Likewise. (varobj_delete): Likewise. (varobj_set_frozen): Likewise. (varobj_get_frozen): Likewise. (varobj_set_value): Likewise. (varobj_update): Likewise. (varobj_editable_p): Likewise. (varobj_floating_p): Likewise. (varobj_has_more): Likewise. (varobj_is_dynamic_p): Likewise. (varobj_default_value_is_changeable_p): Likewise. (varobj_value_is_changeable_p): Likewise. (varobj_is_anonymous_child): Likewise. (varobj_default_is_path_expr_parent): Likewise.
Diffstat (limited to 'gdb/c-varobj.c')
-rw-r--r--gdb/c-varobj.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/gdb/c-varobj.c b/gdb/c-varobj.c
index e3b097b..2517f39 100644
--- a/gdb/c-varobj.c
+++ b/gdb/c-varobj.c
@@ -34,7 +34,7 @@ static void cplus_class_num_children (struct type *type, int children[3]);
This has already been determined by *_describe_child. The easiest
thing to do is to compare the child's name with ANONYMOUS_*_NAME. */
-int
+bool
varobj_is_anonymous_child (const struct varobj *child)
{
return (child->name == ANONYMOUS_STRUCT_NAME
@@ -131,14 +131,14 @@ adjust_value_for_child_access (struct value **value,
/* Is VAR a path expression parent, i.e., can it be used to construct
a valid path expression? */
-static int
+static bool
c_is_path_expr_parent (const struct varobj *var)
{
struct type *type;
/* "Fake" children are not path_expr parents. */
if (CPLUS_FAKE_CHILD (var))
- return 0;
+ return false;
type = varobj_get_gdb_type (var);
@@ -172,10 +172,10 @@ c_is_path_expr_parent (const struct varobj *var)
}
}
- return 0;
+ return false;
}
- return 1;
+ return true;
}
/* C */