From 758cb81029440efad398f1aba4789a18e59daee2 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Sat, 14 Nov 2020 09:44:56 -0700 Subject: [PATCH] Use "bool" in fetch_inferior_event A while back I noticed that fetch_inferior_event used "int" for should_stop, whereas it can be bool. The method it is assigned from: should_stop = thread_fsm->should_stop (thr); ... already returns bool. Tested by rebuilding. gdb/ChangeLog 2020-11-14 Tom Tromey * infrun.c (fetch_inferior_event): Use "bool" for should_stop. --- gdb/ChangeLog | 4 ++++ gdb/infrun.c | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index e06f59c45ba..69af942bc75 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2020-11-14 Tom Tromey + + * infrun.c (fetch_inferior_event): Use "bool" for should_stop. + 2020-11-14 Tom Tromey * opencl-lang.c (opencl_component_ref): Make "comps" const. diff --git a/gdb/infrun.c b/gdb/infrun.c index 990f40aa626..2e5e837452d 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -3914,7 +3914,7 @@ fetch_inferior_event () if (!ecs->wait_some_more) { struct inferior *inf = find_inferior_ptid (ecs->target, ecs->ptid); - int should_stop = 1; + bool should_stop = true; struct thread_info *thr = ecs->event_thread; delete_just_stopped_threads_infrun_breakpoints (); -- 2.30.2