This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
RE: [pushed][PATCH v3 1/4] Extended-remote follow exec
- From: "Breazeal, Don" <Don_Breazeal at mentor dot com>
- To: Pedro Alves <palves at redhat dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Wed, 30 Sep 2015 16:22:23 +0000
- Subject: RE: [pushed][PATCH v3 1/4] Extended-remote follow exec
- Authentication-results: sourceware.org; auth=none
- References: <1441996698-12694-1-git-send-email-donb at codesourcery dot com> <560C0BB8 dot 30804 at redhat dot com>
> -----Original Message-----
> From: Pedro Alves [mailto:palves@redhat.com]
> Sent: Wednesday, September 30, 2015 9:20 AM
> To: Breazeal, Don; gdb-patches@sourceware.org
> Subject: Re: [pushed][PATCH v3 1/4] Extended-remote follow exec
>
> Hi Don,
>
> On 09/11/2015 07:38 PM, Don Breazeal wrote:
> > @@ -5977,6 +6065,7 @@ remote_parse_stop_reply (char *buf, struct
> stop_reply *event)
> > struct remote_arch_state *rsa = get_remote_arch_state ();
> > ULONGEST addr;
> > char *p;
> > + int skipregs = 0;
> >
> > event->ptid = null_ptid;
> > event->rs = get_remote_state ();
> > @@ -6089,11 +6178,42 @@ Packet: '%s'\n"),
> > event->ws.kind = TARGET_WAITKIND_VFORK_DONE;
> > p = skip_to_semicolon (p1 + 1);
> > }
> > + else if (strncmp (p, "exec", p1 - p) == 0)
> > + {
>
> I happened to notice that this is still using strncmp while the rest
> of the magic registers are now using strprefix instead (26d56a939e).
> Looks like a mid-air collision happened.
> This one could be adjusted to use strprefix too, right?
Absolutely, thanks for pointing it out. I will take care of that today.
--Don