Hi Simon,

You are absolutely right. I am new to the community and I noticed that I
didn't merge two of my commits together in the patch file. I have already
merged them and included in the attachment in this email. I also uploaded
it to the bug report:
https://sourceware.org/bugzilla/attachment.cgi?id=13953

Best,
Louis


On Wed, 2 Feb 2022 at 15:57, Simon Marchi <simark@simark.ca> wrote:

> On 2022-02-02 2:31 p.m., siweihe.eng--- via Gdb-patches wrote:
> > From: Louis-He <1726110778@qq.com>
> >
> > After this fix, the possibility of successful run on mac os increases.
> However, it is still possible that the gdb hangs. A workaround is issue
> "control+c" and the gdb will continue running as expected. The root cause
> of the bug hasn't been identified, and this fix is a temporary patch to
> PR-24069.
> > ---
> >  gdb/darwin-nat.c | 10 +++++-----
> >  1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/gdb/darwin-nat.c b/gdb/darwin-nat.c
> > index bba6acc7ea1..844dbf499fe 100644
> > --- a/gdb/darwin-nat.c
> > +++ b/gdb/darwin-nat.c
> > @@ -1104,16 +1104,16 @@ darwin_nat_target::decode_message
> (mach_msg_header_t *hdr,
> >               }
> >
> >             if (WIFEXITED (wstatus))
> > -    {
> > -      status->set_exited (WEXITSTATUS (wstatus));
> > -    }
> > +             {
> > +               status->set_exited (WEXITSTATUS (wstatus));
> > +             }
> >             else if (WIFSTOPPED (wstatus))
> >               {
> > -      status->set_ignore ();
> > +               status->set_ignore ();
> >                 inferior_debug (4, _("darwin_wait: pid %d received
> WIFSTOPPED\n"), res_pid);
> >                 return minus_one_ptid;
> >               }
> > -        else
> > +           else
> >               {
> >                 status->set_signalled
> >                   (gdb_signal_from_host (WTERMSIG (wstatus)));
> > --
> > 2.32.0 (Apple Git-132)
> >
>
>
> Hi,
>
> You patch doesn't seem to change anything, except whitespaces, I guess
> that's not
> what you meant to send.
>
> Simon
>