aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Burgess <aburgess@redhat.com>2022-10-17 15:01:24 +0100
committerAndrew Burgess <aburgess@redhat.com>2023-01-13 16:34:10 +0000
commit8dd08de7e48850496321548099ef86d3275679b9 (patch)
tree1efd0eec7916a9d02d18b645d8d36da7e09752a9
parent6f9f448118eaeaf006f867a25699aef7d8c72770 (diff)
downloadfsf-binutils-gdb-8dd08de7e48850496321548099ef86d3275679b9.zip
fsf-binutils-gdb-8dd08de7e48850496321548099ef86d3275679b9.tar.gz
fsf-binutils-gdb-8dd08de7e48850496321548099ef86d3275679b9.tar.bz2
gdb: int to bool conversion for normal_stop
Change the return type of normal_stop (infrun.c) from int to bool. Update callers. I've also converted the (void) to () in the function declaration and definition, given I was changing those lines anyway. There should be no user visible changes after this commit.
-rw-r--r--gdb/infcmd.c4
-rw-r--r--gdb/infrun.c10
-rw-r--r--gdb/infrun.h2
3 files changed, 7 insertions, 9 deletions
diff --git a/gdb/infcmd.c b/gdb/infcmd.c
index 0497ad0..7d5ec77 100644
--- a/gdb/infcmd.c
+++ b/gdb/infcmd.c
@@ -885,12 +885,10 @@ step_1 (int skip_subroutines, int single_inst, const char *count_string)
proceed ((CORE_ADDR) -1, GDB_SIGNAL_DEFAULT);
else
{
- int proceeded;
-
/* Stepped into an inline frame. Pretend that we've
stopped. */
thr->thread_fsm ()->clean_up (thr);
- proceeded = normal_stop ();
+ bool proceeded = normal_stop ();
if (!proceeded)
inferior_event_handler (INF_EXEC_COMPLETE);
all_uis_check_sync_execution_done ();
diff --git a/gdb/infrun.c b/gdb/infrun.c
index 181d961..1a2ef2d 100644
--- a/gdb/infrun.c
+++ b/gdb/infrun.c
@@ -4219,7 +4219,7 @@ fetch_inferior_event ()
else
{
bool should_notify_stop = true;
- int proceeded = 0;
+ bool proceeded = false;
stop_all_threads_if_all_stop_mode ();
@@ -8582,8 +8582,8 @@ stop_context::changed () const
/* See infrun.h. */
-int
-normal_stop (void)
+bool
+normal_stop ()
{
struct target_waitstatus last;
@@ -8742,7 +8742,7 @@ normal_stop (void)
the observers would print a stop for the wrong
thread/inferior. */
if (saved_context.changed ())
- return 1;
+ return true;
/* Notify observers about the stop. This is where the interpreters
print the stop event. */
@@ -8764,7 +8764,7 @@ normal_stop (void)
breakpoint_auto_delete (inferior_thread ()->control.stop_bpstat);
}
- return 0;
+ return false;
}
int
diff --git a/gdb/infrun.h b/gdb/infrun.h
index 19e6815..43fd1b4 100644
--- a/gdb/infrun.h
+++ b/gdb/infrun.h
@@ -149,7 +149,7 @@ extern process_stratum_target *user_visible_resume_target (ptid_t resume_ptid);
appropriate messages, remove breakpoints, give terminal our modes,
and run the stop hook. Returns true if the stop hook proceeded the
target, false otherwise. */
-extern int normal_stop (void);
+extern bool normal_stop ();
/* Return the cached copy of the last target/ptid/waitstatus returned
by target_wait(). The data is actually cached by handle_inferior_event(),