From: Daniel Jacobowitz <drow@mvista.com>
To: Joel Brobecker <brobecker@gnat.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFA/testsuite] test hand function call in commands list
Date: Sun, 29 Jun 2003 20:11:00 -0000 [thread overview]
Message-ID: <20030629201100.GB32279@nevyn.them.org> (raw)
In-Reply-To: <20030414154048.GC1151@gnat.com>
On Mon, Apr 14, 2003 at 11:40:48AM -0400, Joel Brobecker wrote:
> The attached patch to break.exp was written by Klee Dienes and
> submitted in the following message:
>
> http://sources.redhat.com/ml/gdb-patches/2002-10/msg00586.html
>
> The purpose of the test is to make sure that a problem occuring with
> function calls inside a commands list does not cause a SIGSEGV.
>
> 2003-04-14 J. Brobecker <brobecker@gnat.com>
>
> From Klee Dienne <kdienes@apple.com>
> * gdb.base/break.exp: Add multiple calls to the inferior in
> the user-commands for 'break 79'. Add a check for the calls
> to the inferior in the check for the result. Add new test, to
> check that user-defined breakpoint commands are called for functions
> called by the user (currently fails).
>
> When I ran the test, I was very surprised to see one FAIL. I am
> describing below the symptoms (no time to investigate this today), but I
> think the test is worthwhile adding anyway. Ok to apply?
This patch got sidetracked in a discussion of the one FAIL. Since
it seems that the "failure" isn't something broken in GDB, but rather a
behavioral decision, I'd rather not add that part of the test.
I was going to separate out and apply the remainder of the patch, but
it's not acceptable - it tries to match inferior output directly, which
will fail for remote testing. If someone wants to rewrite this test to
call inferior functions that set variables, and then print the
variables to verify that the functions were called, that'd be great.
--
Daniel Jacobowitz
MontaVista Software Debian GNU/Linux Developer
prev parent reply other threads:[~2003-06-29 20:11 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-04-14 15:40 Joel Brobecker
2003-04-16 14:23 ` Daniel Jacobowitz
2003-04-16 23:09 ` Joel Brobecker
2003-04-17 2:40 ` Daniel Jacobowitz
2003-04-23 0:50 ` Joel Brobecker
2003-04-23 2:42 ` Michael Snyder
2003-06-29 20:11 ` Daniel Jacobowitz [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20030629201100.GB32279@nevyn.them.org \
--to=drow@mvista.com \
--cc=brobecker@gnat.com \
--cc=gdb-patches@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox