Browse Source

gdb/remote: Use true/false instead of 1/0

The remote_state::starting_up member variable is already of type bool,
but in some places we still write to it using 1 and 0.  This commit
just updates things to use true and false.

There should be no user visible change after this commit.

gdb/ChangeLog:

	* remote.c (remote_target::start_remote): Set 'starting_up' using
	boolean values instead of integers.
binutils-2_37-branch
Andrew Burgess 5 years ago
parent
commit
4d60b89770
  1. 5
      gdb/ChangeLog
  2. 8
      gdb/remote.c

5
gdb/ChangeLog

@ -1,3 +1,8 @@
2021-06-28 Andrew Burgess <andrew.burgess@embecosm.com>
* remote.c (remote_target::start_remote): Set 'starting_up' using
boolean values instead of integers.
2021-06-25 Simon Marchi <simon.marchi@polymtl.ca>
* guile/scm-param.c (struct param_smob) <set_command,

8
gdb/remote.c

@ -4667,7 +4667,7 @@ remote_target::start_remote (int from_tty, int extended_p)
Ctrl-C before we're connected and synced up can't interrupt the
target. Instead, it offers to drop the (potentially wedged)
connection. */
rs->starting_up = 1;
rs->starting_up = true;
QUIT;
@ -4808,7 +4808,7 @@ remote_target::start_remote (int from_tty, int extended_p)
/* We're connected, but not running. Drop out before we
call start_remote. */
rs->starting_up = 0;
rs->starting_up = false;
return;
}
else
@ -4923,7 +4923,7 @@ remote_target::start_remote (int from_tty, int extended_p)
/* We're connected, but not running. Drop out before we
call start_remote. */
rs->starting_up = 0;
rs->starting_up = false;
return;
}
@ -4968,7 +4968,7 @@ remote_target::start_remote (int from_tty, int extended_p)
target, our symbols have been relocated, and we're merged the
target's tracepoints with ours. We're done with basic start
up. */
rs->starting_up = 0;
rs->starting_up = false;
/* Maybe breakpoints are global and need to be inserted now. */
if (breakpoints_should_be_inserted_now ())

Loading…
Cancel
Save