diff options
author | Simon Marchi <simon.marchi@ericsson.com> | 2017-11-23 11:00:56 -0500 |
---|---|---|
committer | Simon Marchi <simon.marchi@ericsson.com> | 2017-11-23 11:00:56 -0500 |
commit | 4c37490d92c0168e5192ce109a791c4b3fec9f8e (patch) | |
tree | 51ca155671475582d88cc8cb52f8d4259a4b5813 /gdb/c-varobj.c | |
parent | eb02c04dc3600e18bb06dc1ef8fc3cf9a967d550 (diff) | |
download | gdb-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.c | 10 |
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 */ |