aboutsummaryrefslogtreecommitdiff
path: root/gdb/infrun.c
diff options
context:
space:
mode:
authorSimon Marchi <simon.marchi@efficios.com>2020-02-03 23:02:28 -0500
committerSimon Marchi <simon.marchi@polymtl.ca>2020-02-03 23:06:55 -0500
commit719546c44f5777a3902a2f913c70fd15b942461d (patch)
tree6f4b7f34edaa5acdba52295cfaec610404bdc2e3 /gdb/infrun.c
parentc5d7be0c97a75ff20b49d57bff078360464125fb (diff)
downloadgdb-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.c24
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);
}