Re: [patch] fix for PR2424

Daniel Jacobowitz
2008-04-15T14:57:16Z

On Tue, Apr 15, 2008 at 10:43:47AM -0400, Aleksandar Ristovski wrote:
> NOTE2: Due to quite significant rework of certain functions in
> mi-support.exp,
> committed diff for testsuite is not exactly the same as the one I posted. I
> took
> liberty to commit anyway since the changes are functionally the same; note
> the
> slightly awkward syntax for "mi_expect_stop" call, namely the "extra"
> argument
> which now must be something like { "" "disp=\"keep\"" } - I don't think it's
> too
> bad (it can be beautified later).

Committing anyway is fine. When you commit a modified version of a
patch, though, please post the final version for the list archives.
Thanks. That way anyone whose code is affected can see the changes.

> NOTE3: In mi-support.exp (mi_expect_stop), I put argument "after_reason" to
> be
> after the reason (I couldn't find it being used anywhere anyway, so this
> should
> be a safe change).

Vladimir presumably has plans for it :-)

> NOTE4: Documentation is not updated (I will try to do that now and submit
> the
> diff for approval).

Thanks.

--
Daniel Jacobowitz
CodeSourcery