Re: [patch] gdb.base/pending.exp replace gdb_test_multi

[Michael Snyder <msnyder at vmware dot com>]

Jan Kratochvil wrote:
> On Sat, 05 Jun 2010 00:16:42 +0200, Michael Snyder wrote:
> > This is a simplification.
>
> > -gdb_test_multiple "break pendfunc2" "Don't set pending breakpoint" {
> > -     -re ".*Make breakpoint pending.*y or \\\[n\\\]. $" {
> > -	    gdb_test "n" "" "Don't set pending breakpoint"
> > -     }
> > -}
> > +gdb_test "break pendfunc2" \
> > +    "" \
> > +    "Don't set pending breakpoint" \
> > +    ".*Make breakpoint pending.*y or \\\[n\\\]. $" \
> > +    "n"
>
> With ".*" vs. gdb_test_no_output distinguishing now should be ".*" here
> instead?  (Maybe a more specific string would be also appropriate.)


No, unfortunately, the expected string here is the empty string;
if we had said "y" to the query, then there would be some output
for us to look for.

Believe me, it's almost heart-breaking for me to have to leave this,
but I don't see it as worthwhile to extend gdb_test_no_output to
handle the query case.

Michael