* [PATCH] gdb/remote.c: Address conflicting enum and method name
@ 2021-01-20 23:29 Joel Sherrill
2021-01-21 1:41 ` Simon Marchi via Gdb-patches
0 siblings, 1 reply; 3+ messages in thread
From: Joel Sherrill @ 2021-01-20 23:29 UTC (permalink / raw)
To: gdb-patches; +Cc: Joel Sherrill
---
gdb/remote.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/gdb/remote.c b/gdb/remote.c
index b4c6fc2108..5daabf236b 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -1104,7 +1104,7 @@ struct remote_thread_info : public private_thread_info
CORE_ADDR watch_data_address = 0;
/* Get the thread's resume state. */
- enum resume_state resume_state () const
+ enum resume_state get_resume_state () const
{
return m_resume_state;
}
@@ -6393,7 +6393,7 @@ remote_target::resume (ptid_t ptid, int step, enum gdb_signal siggnal)
/* We don't expect the core to ask to resume an already resumed (from
its point of view) thread. */
- gdb_assert (remote_thr->resume_state () == resume_state::NOT_RESUMED);
+ gdb_assert (remote_thr->get_resume_state () == resume_state::NOT_RESUMED);
remote_thr->set_resumed_pending_vcont (step, siggnal);
return;
@@ -6645,7 +6645,7 @@ remote_target::commit_resume ()
/* If a thread of a process is not meant to be resumed, then we
can't wildcard that process. */
- if (priv->resume_state () == resume_state::NOT_RESUMED)
+ if (priv->get_resume_state () == resume_state::NOT_RESUMED)
{
get_remote_inferior (tp->inf)->may_wildcard_vcont = false;
@@ -6677,7 +6677,7 @@ remote_target::commit_resume ()
/* If the thread was previously vCont-resumed, no need to send a specific
action for it. If we didn't receive a resume request for it, don't
send an action for it either. */
- if (remote_thr->resume_state () != resume_state::RESUMED_PENDING_VCONT)
+ if (remote_thr->get_resume_state () != resume_state::RESUMED_PENDING_VCONT)
continue;
gdb_assert (!thread_is_in_step_over_chain (tp));
@@ -7803,7 +7803,7 @@ remote_target::select_thread_for_ambiguous_stop_reply
{
remote_thread_info *remote_thr = get_remote_thread_info (thr);
- if (remote_thr->resume_state () != resume_state::RESUMED)
+ if (remote_thr->get_resume_state () != resume_state::RESUMED)
continue;
if (first_resumed_thread == nullptr)
--
2.24.3
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] gdb/remote.c: Address conflicting enum and method name
2021-01-20 23:29 [PATCH] gdb/remote.c: Address conflicting enum and method name Joel Sherrill
@ 2021-01-21 1:41 ` Simon Marchi via Gdb-patches
2021-01-21 2:19 ` Joel Sherrill
0 siblings, 1 reply; 3+ messages in thread
From: Simon Marchi via Gdb-patches @ 2021-01-21 1:41 UTC (permalink / raw)
To: Joel Sherrill, gdb-patches
Thanks, I wrote a commit message and ChangeLog entry and pushed it.
Simon
On 2021-01-20 6:29 p.m., Joel Sherrill wrote:
> ---
> gdb/remote.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/gdb/remote.c b/gdb/remote.c
> index b4c6fc2108..5daabf236b 100644
> --- a/gdb/remote.c
> +++ b/gdb/remote.c
> @@ -1104,7 +1104,7 @@ struct remote_thread_info : public private_thread_info
> CORE_ADDR watch_data_address = 0;
>
> /* Get the thread's resume state. */
> - enum resume_state resume_state () const
> + enum resume_state get_resume_state () const
> {
> return m_resume_state;
> }
> @@ -6393,7 +6393,7 @@ remote_target::resume (ptid_t ptid, int step, enum gdb_signal siggnal)
>
> /* We don't expect the core to ask to resume an already resumed (from
> its point of view) thread. */
> - gdb_assert (remote_thr->resume_state () == resume_state::NOT_RESUMED);
> + gdb_assert (remote_thr->get_resume_state () == resume_state::NOT_RESUMED);
>
> remote_thr->set_resumed_pending_vcont (step, siggnal);
> return;
> @@ -6645,7 +6645,7 @@ remote_target::commit_resume ()
>
> /* If a thread of a process is not meant to be resumed, then we
> can't wildcard that process. */
> - if (priv->resume_state () == resume_state::NOT_RESUMED)
> + if (priv->get_resume_state () == resume_state::NOT_RESUMED)
> {
> get_remote_inferior (tp->inf)->may_wildcard_vcont = false;
>
> @@ -6677,7 +6677,7 @@ remote_target::commit_resume ()
> /* If the thread was previously vCont-resumed, no need to send a specific
> action for it. If we didn't receive a resume request for it, don't
> send an action for it either. */
> - if (remote_thr->resume_state () != resume_state::RESUMED_PENDING_VCONT)
> + if (remote_thr->get_resume_state () != resume_state::RESUMED_PENDING_VCONT)
> continue;
>
> gdb_assert (!thread_is_in_step_over_chain (tp));
> @@ -7803,7 +7803,7 @@ remote_target::select_thread_for_ambiguous_stop_reply
> {
> remote_thread_info *remote_thr = get_remote_thread_info (thr);
>
> - if (remote_thr->resume_state () != resume_state::RESUMED)
> + if (remote_thr->get_resume_state () != resume_state::RESUMED)
> continue;
>
> if (first_resumed_thread == nullptr)
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] gdb/remote.c: Address conflicting enum and method name
2021-01-21 1:41 ` Simon Marchi via Gdb-patches
@ 2021-01-21 2:19 ` Joel Sherrill
0 siblings, 0 replies; 3+ messages in thread
From: Joel Sherrill @ 2021-01-21 2:19 UTC (permalink / raw)
To: Simon Marchi; +Cc: GDB patches
On Wed, Jan 20, 2021, 7:41 PM Simon Marchi <simon.marchi@polymtl.ca> wrote:
> Thanks, I wrote a commit message and ChangeLog entry and pushed it.
>
Thanks. This was too easy to stay broken.
I will close the PR if it didn't get picked up
>
> Simon
>
> On 2021-01-20 6:29 p.m., Joel Sherrill wrote:
> > ---
> > gdb/remote.c | 10 +++++-----
> > 1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/gdb/remote.c b/gdb/remote.c
> > index b4c6fc2108..5daabf236b 100644
> > --- a/gdb/remote.c
> > +++ b/gdb/remote.c
> > @@ -1104,7 +1104,7 @@ struct remote_thread_info : public
> private_thread_info
> > CORE_ADDR watch_data_address = 0;
> >
> > /* Get the thread's resume state. */
> > - enum resume_state resume_state () const
> > + enum resume_state get_resume_state () const
> > {
> > return m_resume_state;
> > }
> > @@ -6393,7 +6393,7 @@ remote_target::resume (ptid_t ptid, int step, enum
> gdb_signal siggnal)
> >
> > /* We don't expect the core to ask to resume an already resumed
> (from
> > its point of view) thread. */
> > - gdb_assert (remote_thr->resume_state () ==
> resume_state::NOT_RESUMED);
> > + gdb_assert (remote_thr->get_resume_state () ==
> resume_state::NOT_RESUMED);
> >
> > remote_thr->set_resumed_pending_vcont (step, siggnal);
> > return;
> > @@ -6645,7 +6645,7 @@ remote_target::commit_resume ()
> >
> > /* If a thread of a process is not meant to be resumed, then we
> > can't wildcard that process. */
> > - if (priv->resume_state () == resume_state::NOT_RESUMED)
> > + if (priv->get_resume_state () == resume_state::NOT_RESUMED)
> > {
> > get_remote_inferior (tp->inf)->may_wildcard_vcont = false;
> >
> > @@ -6677,7 +6677,7 @@ remote_target::commit_resume ()
> > /* If the thread was previously vCont-resumed, no need to send a
> specific
> > action for it. If we didn't receive a resume request for it,
> don't
> > send an action for it either. */
> > - if (remote_thr->resume_state () !=
> resume_state::RESUMED_PENDING_VCONT)
> > + if (remote_thr->get_resume_state () !=
> resume_state::RESUMED_PENDING_VCONT)
> > continue;
> >
> > gdb_assert (!thread_is_in_step_over_chain (tp));
> > @@ -7803,7 +7803,7 @@
> remote_target::select_thread_for_ambiguous_stop_reply
> > {
> > remote_thread_info *remote_thr = get_remote_thread_info (thr);
> >
> > - if (remote_thr->resume_state () != resume_state::RESUMED)
> > + if (remote_thr->get_resume_state () != resume_state::RESUMED)
> > continue;
> >
> > if (first_resumed_thread == nullptr)
> >
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-01-21 2:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-20 23:29 [PATCH] gdb/remote.c: Address conflicting enum and method name Joel Sherrill
2021-01-21 1:41 ` Simon Marchi via Gdb-patches
2021-01-21 2:19 ` Joel Sherrill
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox