Re: [PATCH 3/5] range stepping: gdb

Eli Zaretskii
2013-05-20T19:05:49Z

> Date: Mon, 20 May 2013 19:43:13 +0100
> From: Pedro Alves <palves@redhat.com>
> CC: gdb-patches@sourceware.org
>
> On 05/16/2013 07:08 AM, Eli Zaretskii wrote:
> >> Date: Wed, 15 May 2013 19:20:46 +0100
> >> From: Pedro Alves <palves@redhat.com>
> >> CC: gdb-patches@sourceware.org
> >>
> >> What do you think of this?
> >
> > I think it's a really good and clear description.
> >
> >> I'm okay with removing the whole second paragraph ("If the range is
> >> empty...") if you think the first paragraph is already clear enough.
> >
> > The first paragraph is clear enough, but I find the second paragraph
> > reassuring me in that the interpretation of the first was correct. So
> > I think it is better to leave both.
>
> Sorry for taking long to respond, and letting the context potentially
> slip from you. Here's the full updated patch with that bit merged in.
> Nothing else in the docs changed. I've updated a comment in
> gdbthread.h, giving it a similar clarification treatment.

That's OK with me, thanks.