diff options
author | Tom Tromey <tom@tromey.com> | 2018-06-11 14:35:00 -0600 |
---|---|---|
committer | Tom Tromey <tom@tromey.com> | 2018-07-03 11:36:45 -0600 |
commit | 26a57c9256d7ec2b4da2f1d85a9fba830948dbd9 (patch) | |
tree | 3d6e95a05146b2eea1a317bee32302f3779c86af | |
parent | d2a107e3f9e84808ae0860693f1c488b1e3f5559 (diff) | |
download | gdb-26a57c9256d7ec2b4da2f1d85a9fba830948dbd9.zip gdb-26a57c9256d7ec2b4da2f1d85a9fba830948dbd9.tar.gz gdb-26a57c9256d7ec2b4da2f1d85a9fba830948dbd9.tar.bz2 |
Remove ptid_match
This removes ptid_match in favor of the ptid_t::matches method.
gdb/ChangeLog
2018-07-03 Tom Tromey <tom@tromey.com>
* common/ptid.c (ptid_match): Remove.
* common/ptid.h (ptid_match): Don't declare.
* fbsd-nat.c: Update.
* infcmd.c: Update.
* infrun.c: Update.
* linux-nat.c: Update.
* record-btrace.c: Update.
* regcache.c: Update.
* remote.c: Update.
gdb/gdbserver/ChangeLog
2018-07-03 Tom Tromey <tom@tromey.com>
* server.c: Update.
-rw-r--r-- | gdb/ChangeLog | 12 | ||||
-rw-r--r-- | gdb/common/ptid.c | 8 | ||||
-rw-r--r-- | gdb/common/ptid.h | 4 | ||||
-rw-r--r-- | gdb/fbsd-nat.c | 2 | ||||
-rw-r--r-- | gdb/gdbserver/ChangeLog | 4 | ||||
-rw-r--r-- | gdb/gdbserver/server.c | 6 | ||||
-rw-r--r-- | gdb/infcmd.c | 2 | ||||
-rw-r--r-- | gdb/infrun.c | 12 | ||||
-rw-r--r-- | gdb/linux-nat.c | 4 | ||||
-rw-r--r-- | gdb/record-btrace.c | 14 | ||||
-rw-r--r-- | gdb/regcache.c | 6 | ||||
-rw-r--r-- | gdb/remote.c | 2 |
12 files changed, 40 insertions, 36 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 2072328..e52d898 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,17 @@ 2018-07-03 Tom Tromey <tom@tromey.com> + * common/ptid.c (ptid_match): Remove. + * common/ptid.h (ptid_match): Don't declare. + * fbsd-nat.c: Update. + * infcmd.c: Update. + * infrun.c: Update. + * linux-nat.c: Update. + * record-btrace.c: Update. + * regcache.c: Update. + * remote.c: Update. + +2018-07-03 Tom Tromey <tom@tromey.com> + * common/ptid.c (ptid_tid_p): Remove. * common/ptid.h (ptid_tid_p): Don't declare. * sol-thread.c: Update. diff --git a/gdb/common/ptid.c b/gdb/common/ptid.c index e0f62b1..b9a331a 100644 --- a/gdb/common/ptid.c +++ b/gdb/common/ptid.c @@ -32,11 +32,3 @@ ptid_equal (const ptid_t &ptid1, const ptid_t &ptid2) { return ptid1 == ptid2; } - -/* See ptid.h. */ - -int -ptid_match (const ptid_t &ptid, const ptid_t &filter) -{ - return ptid.matches (filter); -} diff --git a/gdb/common/ptid.h b/gdb/common/ptid.h index 86878de..2fc84b9 100644 --- a/gdb/common/ptid.h +++ b/gdb/common/ptid.h @@ -160,8 +160,4 @@ extern ptid_t minus_one_ptid; extern int ptid_equal (const ptid_t &ptid1, const ptid_t &ptid2); -/* See ptid_t::matches. */ - -extern int ptid_match (const ptid_t &ptid, const ptid_t &filter); - #endif diff --git a/gdb/fbsd-nat.c b/gdb/fbsd-nat.c index 4967dcb..4542887 100644 --- a/gdb/fbsd-nat.c +++ b/gdb/fbsd-nat.c @@ -1130,7 +1130,7 @@ fbsd_nat_target::resume (ptid_t ptid, int step, enum gdb_signal signo) ALL_NON_EXITED_THREADS (tp) { - if (!ptid_match (tp->ptid, ptid)) + if (!tp->ptid.matches (ptid)) continue; if (ptrace (PT_RESUME, tp->ptid.lwp (), NULL, 0) == -1) diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog index 52335cf..5e2d64e 100644 --- a/gdb/gdbserver/ChangeLog +++ b/gdb/gdbserver/ChangeLog @@ -1,5 +1,9 @@ 2018-07-03 Tom Tromey <tom@tromey.com> + * server.c: Update. + +2018-07-03 Tom Tromey <tom@tromey.com> + * linux-low.c: Update. 2018-07-03 Tom Tromey <tom@tromey.com> diff --git a/gdb/gdbserver/server.c b/gdb/gdbserver/server.c index 5c24315..3c58c6e 100644 --- a/gdb/gdbserver/server.c +++ b/gdb/gdbserver/server.c @@ -190,7 +190,7 @@ remove_all_on_match_ptid (QUEUE (notif_event_p) *q, ptid_t filter_ptid = *(ptid_t *) data; struct vstop_notif *vstop_event = (struct vstop_notif *) event; - if (ptid_match (vstop_event->ptid, filter_ptid)) + if (vstop_event->ptid.matches (filter_ptid)) { if (q->free_func != NULL) q->free_func (event); @@ -229,13 +229,13 @@ in_queued_stop_replies_ptid (QUEUE (notif_event_p) *q, ptid_t filter_ptid = *(ptid_t *) data; struct vstop_notif *vstop_event = (struct vstop_notif *) event; - if (ptid_match (vstop_event->ptid, filter_ptid)) + if (vstop_event->ptid.matches (filter_ptid)) return 0; /* Don't resume fork children that GDB does not know about yet. */ if ((vstop_event->status.kind == TARGET_WAITKIND_FORKED || vstop_event->status.kind == TARGET_WAITKIND_VFORKED) - && ptid_match (vstop_event->status.value.related_pid, filter_ptid)) + && vstop_event->status.value.related_pid.matches (filter_ptid)) return 0; return 1; diff --git a/gdb/infcmd.c b/gdb/infcmd.c index 41322c6..c9c70fb 100644 --- a/gdb/infcmd.c +++ b/gdb/infcmd.c @@ -1351,7 +1351,7 @@ signal_command (const char *signum_exp, int from_tty) { if (ptid_equal (tp->ptid, inferior_ptid)) continue; - if (!ptid_match (tp->ptid, resume_ptid)) + if (!tp->ptid.matches (resume_ptid)) continue; if (tp->suspend.stop_signal != GDB_SIGNAL_0 diff --git a/gdb/infrun.c b/gdb/infrun.c index c1a0cad..c1214ee 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -2877,7 +2877,7 @@ clear_proceed_status (int step) we're about to resume, implicitly and explicitly. */ ALL_NON_EXITED_THREADS (tp) { - if (!ptid_match (tp->ptid, resume_ptid)) + if (!tp->ptid.matches (resume_ptid)) continue; clear_proceed_status_thread (tp); } @@ -3095,7 +3095,7 @@ proceed (CORE_ADDR addr, enum gdb_signal siggnal) continue; /* Ignore threads of processes we're not resuming. */ - if (!ptid_match (tp->ptid, resume_ptid)) + if (!tp->ptid.matches (resume_ptid)) continue; if (!thread_still_needs_step_over (tp)) @@ -3149,7 +3149,7 @@ proceed (CORE_ADDR addr, enum gdb_signal siggnal) ALL_NON_EXITED_THREADS (tp) { /* Ignore threads of processes we're not resuming. */ - if (!ptid_match (tp->ptid, resume_ptid)) + if (!tp->ptid.matches (resume_ptid)) continue; if (tp->resumed) @@ -3291,7 +3291,7 @@ infrun_thread_stop_requested (ptid_t ptid) thread had been temporarily paused for some step-over), set up for reporting the stop now. */ ALL_NON_EXITED_THREADS (tp) - if (ptid_match (tp->ptid, ptid)) + if (tp->ptid.matches (ptid)) { if (tp->state != THREAD_RUNNING) continue; @@ -3453,7 +3453,7 @@ random_pending_event_thread (ptid_t waiton_ptid) /* First see how many events we have. Count only resumed threads that have an event pending. */ ALL_NON_EXITED_THREADS (event_tp) - if (ptid_match (event_tp->ptid, waiton_ptid) + if (event_tp->ptid.matches (waiton_ptid) && event_tp->resumed && event_tp->suspend.waitstatus_pending_p) num_events++; @@ -3472,7 +3472,7 @@ random_pending_event_thread (ptid_t waiton_ptid) /* Select the Nth thread that has had an event. */ ALL_NON_EXITED_THREADS (event_tp) - if (ptid_match (event_tp->ptid, waiton_ptid) + if (event_tp->ptid.matches (waiton_ptid) && event_tp->resumed && event_tp->suspend.waitstatus_pending_p) if (random_selector-- == 0) diff --git a/gdb/linux-nat.c b/gdb/linux-nat.c index 1c12240..c79d72c 100644 --- a/gdb/linux-nat.c +++ b/gdb/linux-nat.c @@ -958,7 +958,7 @@ iterate_over_lwps (ptid_t filter, { lpnext = lp->next; - if (ptid_match (lp->ptid, filter)) + if (lp->ptid.matches (filter)) { if ((*callback) (lp, data) != 0) return lp; @@ -3513,7 +3513,7 @@ resume_stopped_resumed_lwps (struct lwp_info *lp, void *data) /* Don't bother if there's a breakpoint at PC that we'd hit immediately, and we're not waiting for this LWP. */ - if (!ptid_match (lp->ptid, *wait_ptid_p)) + if (!lp->ptid.matches (*wait_ptid_p)) { if (breakpoint_inserted_here_p (regcache->aspace (), pc)) leave_stopped = 1; diff --git a/gdb/record-btrace.c b/gdb/record-btrace.c index b67d711..09a3f6c 100644 --- a/gdb/record-btrace.c +++ b/gdb/record-btrace.c @@ -1401,7 +1401,7 @@ record_btrace_target::record_is_replaying (ptid_t ptid) struct thread_info *tp; ALL_NON_EXITED_THREADS (tp) - if (ptid_match (tp->ptid, ptid) && btrace_is_replaying (tp)) + if (tp->ptid.matches (ptid) && btrace_is_replaying (tp)) return true; return false; @@ -2178,12 +2178,12 @@ record_btrace_target::resume (ptid_t ptid, int step, enum gdb_signal signal) For all-stop targets, we only step INFERIOR_PTID and continue others. */ if (!target_is_non_stop_p ()) { - gdb_assert (ptid_match (inferior_ptid, ptid)); + gdb_assert (inferior_ptid.matches (ptid)); ALL_NON_EXITED_THREADS (tp) - if (ptid_match (tp->ptid, ptid)) + if (tp->ptid.matches (ptid)) { - if (ptid_match (tp->ptid, inferior_ptid)) + if (tp->ptid.matches (inferior_ptid)) record_btrace_resume_thread (tp, flag); else record_btrace_resume_thread (tp, cflag); @@ -2192,7 +2192,7 @@ record_btrace_target::resume (ptid_t ptid, int step, enum gdb_signal signal) else { ALL_NON_EXITED_THREADS (tp) - if (ptid_match (tp->ptid, ptid)) + if (tp->ptid.matches (ptid)) record_btrace_resume_thread (tp, flag); } @@ -2555,7 +2555,7 @@ record_btrace_target::wait (ptid_t ptid, struct target_waitstatus *status, ALL_NON_EXITED_THREADS (tp) { - if (ptid_match (tp->ptid, ptid) + if (tp->ptid.matches (ptid) && ((tp->btrace.flags & (BTHR_MOVE | BTHR_STOP)) != 0)) moving.push_back (tp); } @@ -2682,7 +2682,7 @@ record_btrace_target::stop (ptid_t ptid) struct thread_info *tp; ALL_NON_EXITED_THREADS (tp) - if (ptid_match (tp->ptid, ptid)) + if (tp->ptid.matches (ptid)) { tp->btrace.flags &= ~BTHR_MOVE; tp->btrace.flags |= BTHR_STOP; diff --git a/gdb/regcache.c b/gdb/regcache.c index 1e241de..bc916f3 100644 --- a/gdb/regcache.c +++ b/gdb/regcache.c @@ -452,7 +452,7 @@ registers_changed_ptid (ptid_t ptid) it != regcache::current_regcache.end (); ) { - if (ptid_match ((*it)->ptid (), ptid)) + if ((*it)->ptid ().matches (ptid)) { delete *it; it = regcache::current_regcache.erase_after (oit); @@ -461,13 +461,13 @@ registers_changed_ptid (ptid_t ptid) oit = it++; } - if (ptid_match (current_thread_ptid, ptid)) + if (current_thread_ptid.matches (ptid)) { current_thread_ptid = null_ptid; current_thread_arch = NULL; } - if (ptid_match (inferior_ptid, ptid)) + if (inferior_ptid.matches (ptid)) { /* We just deleted the regcache of the current thread. Need to forget about any frames we have cached, too. */ diff --git a/gdb/remote.c b/gdb/remote.c index 8343058..c72060f 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -6075,7 +6075,7 @@ remote_target::append_pending_thread_resumptions (char *p, char *endp, struct thread_info *thread; ALL_NON_EXITED_THREADS (thread) - if (ptid_match (thread->ptid, ptid) + if (thread->ptid.matches (ptid) && !ptid_equal (inferior_ptid, thread->ptid) && thread->suspend.stop_signal != GDB_SIGNAL_0) { |