diff options
author | Simon Marchi <simon.marchi@efficios.com> | 2020-02-03 23:02:28 -0500 |
---|---|---|
committer | Simon Marchi <simon.marchi@polymtl.ca> | 2020-02-03 23:06:55 -0500 |
commit | 719546c44f5777a3902a2f913c70fd15b942461d (patch) | |
tree | 6f4b7f34edaa5acdba52295cfaec610404bdc2e3 /gdb/infrun.c | |
parent | c5d7be0c97a75ff20b49d57bff078360464125fb (diff) | |
download | gdb-719546c44f5777a3902a2f913c70fd15b942461d.zip gdb-719546c44f5777a3902a2f913c70fd15b942461d.tar.gz gdb-719546c44f5777a3902a2f913c70fd15b942461d.tar.bz2 |
Change ints to bools around thread_info executing/resumed
Switch thread_info::resumed to bool (thread_info::executing already is a bool),
and try to change everything more or less related to that to consistently use
true/false instead of 1/0.
gdb/ChangeLog:
* fork-child.c (gdb_startup_inferior): Use bool instead of int.
* gdbthread.h (class thread_info) <resumed>: Likewise.
* infrun.c (resume_1): Likewise.
(proceed): Likewise.
(infrun_thread_stop_requested): Likewise.
(stop_all_threads): Likewise.
(handle_inferior_event): Likewise.
(restart_threads): Likewise.
(finish_step_over): Likewise.
(keep_going_stepped_thread): Likewise.
* linux-nat.c (attach_proc_task_lwp_callback): Likewise.
(linux_handle_extended_wait): Likewise.
* record-btrace.c (get_thread_current_frame_id): Likewise.
* record-full.c (record_full_wait_1): Likewise.
* remote.c (remote_target::process_initial_stop_replies): Likewise.
* target.c (target_resume): Likewise.
* thread.c (set_running_thread): Likewise.
Diffstat (limited to 'gdb/infrun.c')
-rw-r--r-- | gdb/infrun.c | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/gdb/infrun.c b/gdb/infrun.c index c8369cb..3e846f8 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -2282,7 +2282,7 @@ resume_1 (enum gdb_signal sig) } tp->inf->process_target ()->threads_executing = true; - tp->resumed = 1; + tp->resumed = true; /* FIXME: What should we do if we are supposed to resume this thread with a signal? Maybe we should maintain a queue of @@ -2410,7 +2410,7 @@ resume_1 (enum gdb_signal sig) resume_ptid = internal_resume_ptid (user_step); do_target_resume (resume_ptid, 0, GDB_SIGNAL_0); - tp->resumed = 1; + tp->resumed = true; return; } } @@ -2622,7 +2622,7 @@ resume_1 (enum gdb_signal sig) } do_target_resume (resume_ptid, step, sig); - tp->resumed = 1; + tp->resumed = true; } /* Resume the inferior. SIG is the signal to give the inferior @@ -3022,7 +3022,7 @@ proceed (CORE_ADDR addr, enum gdb_signal siggnal) inferior function, as in that case we pretend the inferior doesn't run at all. */ if (!cur_thr->control.in_infcall) - set_running (resume_target, resume_ptid, 1); + set_running (resume_target, resume_ptid, true); if (debug_infrun) fprintf_unfiltered (gdb_stdlog, @@ -3306,7 +3306,7 @@ infrun_thread_stop_requested (ptid_t ptid) /* Otherwise we can process the (new) pending event now. Set it so this pending event is considered by do_target_wait. */ - tp->resumed = 1; + tp->resumed = true; } } @@ -4749,7 +4749,7 @@ stop_all_threads (void) /* The thread may be not executing, but still be resumed with a pending status to process. */ - t->resumed = 0; + t->resumed = false; } } @@ -4788,7 +4788,7 @@ stop_all_threads (void) t->stop_requested = 0; t->executing = 0; - t->resumed = 0; + t->resumed = false; t->control.may_range_step = 0; /* This may be the first time we see the inferior report @@ -5126,10 +5126,10 @@ handle_inferior_event (struct execution_control_state *ecs) else mark_ptid = ecs->ptid; - set_executing (ecs->target, mark_ptid, 0); + set_executing (ecs->target, mark_ptid, false); /* Likewise the resumed flag. */ - set_resumed (ecs->target, mark_ptid, 0); + set_resumed (ecs->target, mark_ptid, false); } switch (ecs->ws.kind) @@ -5623,7 +5623,7 @@ restart_threads (struct thread_info *event_thread) "infrun: restart threads: " "[%s] has pending status\n", target_pid_to_str (tp->ptid).c_str ()); - tp->resumed = 1; + tp->resumed = true; continue; } @@ -5763,7 +5763,7 @@ finish_step_over (struct execution_control_state *ecs) /* This was cleared early, by handle_inferior_event. Set it so this pending event is considered by do_target_wait. */ - tp->resumed = 1; + tp->resumed = true; gdb_assert (!tp->executing); @@ -7424,7 +7424,7 @@ keep_going_stepped_thread (struct thread_info *tp) get_frame_address_space (frame), tp->suspend.stop_pc); - tp->resumed = 1; + tp->resumed = true; resume_ptid = internal_resume_ptid (tp->control.stepping_command); do_target_resume (resume_ptid, 0, GDB_SIGNAL_0); } |