This is the mail archive of the mailing list for the GDB project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] fix for PR2424

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


Daniel Jacobowitz

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]