aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Burgess <aburgess@redhat.com>2025-07-12 11:35:47 +0100
committerAndrew Burgess <aburgess@redhat.com>2025-08-01 10:15:30 +0100
commit7a45c8e03020e8fbe5284e5ffa6d4c710c6fd876 (patch)
tree04ab5f879b4ee814b870ac6da1ea0c7d547d3fc0
parent711667472193c31b67eb3c190d7c91f660ac4bb0 (diff)
downloadbinutils-7a45c8e03020e8fbe5284e5ffa6d4c710c6fd876.zip
binutils-7a45c8e03020e8fbe5284e5ffa6d4c710c6fd876.tar.gz
binutils-7a45c8e03020e8fbe5284e5ffa6d4c710c6fd876.tar.bz2
gdbserver: convert locals to `bool` in captured_main
Within gdbserver/server.cc, in captured_main, convert some locals to bool. Move the declaration of some locals into the body of the function. There should be no user visible changes after this commit. Approved-By: Tom Tromey <tom@tromey.com>
-rw-r--r--gdbserver/server.cc21
1 files changed, 10 insertions, 11 deletions
diff --git a/gdbserver/server.cc b/gdbserver/server.cc
index ce84c7f..a59f941 100644
--- a/gdbserver/server.cc
+++ b/gdbserver/server.cc
@@ -4103,14 +4103,12 @@ static void test_registers_raw_compare_zero_length ()
[[noreturn]] static void
captured_main (int argc, char *argv[])
{
- int bad_attach;
int pid;
char *arg_end;
const char *port = NULL;
char **next_arg = &argv[1];
- volatile int multi_mode = 0;
- volatile int attach = 0;
- int was_running;
+ volatile bool multi_mode = false;
+ volatile bool attach = false;
bool selftest = false;
#if GDB_SELF_TEST
std::vector<const char *> selftest_filters;
@@ -4143,9 +4141,9 @@ captured_main (int argc, char *argv[])
exit (0);
}
else if (strcmp (*next_arg, "--attach") == 0)
- attach = 1;
+ attach = true;
else if (strcmp (*next_arg, "--multi") == 0)
- multi_mode = 1;
+ multi_mode = true;
else if (strcmp (*next_arg, "--wrapper") == 0)
{
char **tmp;
@@ -4328,14 +4326,14 @@ captured_main (int argc, char *argv[])
if (port != NULL)
remote_prepare (port);
- bad_attach = 0;
+ bool bad_attach = false;
pid = 0;
/* --attach used to come after PORT, so allow it there for
compatibility. */
if (*next_arg != NULL && strcmp (*next_arg, "--attach") == 0)
{
- attach = 1;
+ attach = true;
next_arg++;
}
@@ -4345,7 +4343,7 @@ captured_main (int argc, char *argv[])
|| (pid = strtoul (*next_arg, &arg_end, 0)) == 0
|| *arg_end != '\0'
|| next_arg[1] != NULL))
- bad_attach = 1;
+ bad_attach = true;
if (bad_attach)
{
@@ -4410,11 +4408,12 @@ captured_main (int argc, char *argv[])
if (current_thread != nullptr)
current_process ()->dlls_changed = false;
+ bool was_running;
if (cs.last_status.kind () == TARGET_WAITKIND_EXITED
|| cs.last_status.kind () == TARGET_WAITKIND_SIGNALLED)
- was_running = 0;
+ was_running = false;
else
- was_running = 1;
+ was_running = true;
if (!was_running && !multi_mode)
error ("No program to debug");