gdb: use bool in notify_command_param_changed_p and do_set_command
authorSimon Marchi <simon.marchi@polymtl.ca>
Thu, 19 Aug 2021 18:28:59 +0000 (14:28 -0400)
committerSimon Marchi <simon.marchi@polymtl.ca>
Thu, 19 Aug 2021 18:29:30 +0000 (14:29 -0400)
Trivial patch to use bool instead of int.

Change-Id: I9e5f8ee4305272a6671cbaaaf2f0484eff0d1ea5

gdb/cli/cli-setshow.c

index 0290acede7fa35dde53ef2af27dfbea66d95c93e..1025ed2f620d330d3f8e39a5e02ff4b21b38d1ef 100644 (file)
 
 /* Return true if the change of command parameter should be notified.  */
 
-static int
-notify_command_param_changed_p (int param_changed, struct cmd_list_element *c)
+static bool
+notify_command_param_changed_p (bool param_changed, struct cmd_list_element *c)
 {
-  if (param_changed == 0)
-    return 0;
+  if (!param_changed)
+    return false;
 
   if (c->theclass == class_maintenance || c->theclass == class_deprecated
       || c->theclass == class_obscure)
-    return 0;
+    return false;
 
-  return 1;
+  return true;
 }
 
 \f
@@ -305,7 +305,7 @@ void
 do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
 {
   /* A flag to indicate the option is changed or not.  */
-  int option_changed = 0;
+  bool option_changed = false;
 
   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);
            *(char **) c->var = newobj;
 
-           option_changed = 1;
+           option_changed = true;
          }
        else
          xfree (newobj);
@@ -371,7 +371,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
          xfree (*(char **) c->var);
          *(char **) c->var = xstrdup (arg);
 
-         option_changed = 1;
+         option_changed = true;
        }
       break;
     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);
            *(char **) c->var = val;
 
-           option_changed = 1;
+           option_changed = true;
          }
        else
          xfree (val);
@@ -420,7 +420,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
          {
            *(bool *) c->var = val;
 
-           option_changed = 1;
+           option_changed = true;
          }
       }
       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;
 
-           option_changed = 1;
+           option_changed = true;
          }
       }
       break;
@@ -445,7 +445,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
          {
            *(unsigned int *) c->var = val;
 
-           option_changed = 1;
+           option_changed = true;
          }
       }
       break;
@@ -481,7 +481,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
          {
            *(int *) c->var = val;
 
-           option_changed = 1;
+           option_changed = true;
          }
        break;
       }
@@ -499,7 +499,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
          {
            *(const char **) c->var = match;
 
-           option_changed = 1;
+           option_changed = true;
          }
       }
       break;
@@ -510,7 +510,7 @@ do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
        if (*(int *) c->var != val)
          {
            *(int *) c->var = val;
-           option_changed = 1;
+           option_changed = true;
          }
       }
       break;