FreeChainXenon/binutils-gdb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

gdb: use bool in notify_command_param_changed_p and do_set_command

Browse source 
Trivial patch to use bool instead of int.

Change-Id: I9e5f8ee4305272a6671cbaaaf2f0484eff0d1ea5
This commit is contained in:
Simon Marchi 2021-08-19 14:28:59 -04:00
parent ca22cf5ed5
commit 9413ea1609
1 changed files with 16 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

32
gdb/cli/cli-setshow.c
View file

@ -31,17 +31,17 @@
/* Return true if the change of command parameter should be notified. */ /* Return true if the change of command parameter should be notified. */
static int static bool
notify_command_param_changed_p (int param_changed, struct cmd_list_element *c) notify_command_param_changed_p (bool param_changed, struct cmd_list_element *c)
{ {
if (param_changed == 0) if (!param_changed)
return 0; return false;
if (c->theclass == class_maintenance || c->theclass == class_deprecated if (c->theclass == class_maintenance || c->theclass == class_deprecated
|| c->theclass == class_obscure) || c->theclass == class_obscure)
return 0; return false;
return 1; return true;
} }
@ -305,7 +305,7 @@ void
do_set_command (const char *arg, int from_tty, struct cmd_list_element *c) do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
{ {
/* A flag to indicate the option is changed or not. */ /* A flag to indicate the option is changed or not. */
int option_changed = 0; bool option_changed = false;
gdb_assert (c->type == set_cmd); gdb_assert (c->type == set_cmd);
@ -359,7 +359,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
xfree (*(char **) c->var); xfree (*(char **) c->var);
*(char **) c->var = newobj; *(char **) c->var = newobj;
option_changed = 1; option_changed = true;
} }
else else
xfree (newobj); xfree (newobj);
@ -371,7 +371,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
xfree (*(char **) c->var); xfree (*(char **) c->var);
*(char **) c->var = xstrdup (arg); *(char **) c->var = xstrdup (arg);
option_changed = 1; option_changed = true;
} }
break; break;
case var_filename: case var_filename:
@ -404,7 +404,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
xfree (*(char **) c->var); xfree (*(char **) c->var);
*(char **) c->var = val; *(char **) c->var = val;
option_changed = 1; option_changed = true;
} }
else else
xfree (val); xfree (val);
@ -420,7 +420,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
{ {
*(bool *) c->var = val; *(bool *) c->var = val;
option_changed = 1; option_changed = true;
} }
} }
break; break;
@ -432,7 +432,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
{ {
*(enum auto_boolean *) c->var = val; *(enum auto_boolean *) c->var = val;
option_changed = 1; option_changed = true;
} }
} }
break; break;
@ -445,7 +445,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
{ {
*(unsigned int *) c->var = val; *(unsigned int *) c->var = val;
option_changed = 1; option_changed = true;
} }
} }
break; break;
@ -481,7 +481,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
{ {
*(int *) c->var = val; *(int *) c->var = val;
option_changed = 1; option_changed = true;
} }
break; break;
} }
@ -499,7 +499,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
{ {
*(const char **) c->var = match; *(const char **) c->var = match;
option_changed = 1; option_changed = true;
} }
} }
break; break;
@ -510,7 +510,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
if (*(int *) c->var != val) if (*(int *) c->var != val)
{ {
*(int *) c->var = val; *(int *) c->var = val;
option_changed = 1; option_changed = true;
} }
} }
break; break;