Re: [PATCH] gdb/remote.c: Address conflicting enum and method name

[Joel Sherrill <joel@rtems.org>]

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)
> >
>