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

[Jan Kratochvil <jan.kratochvil@redhat.com>]

On Tue, 08 Jun 2010 01:00:03 +0200, Michael Snyder wrote:
> Jan Kratochvil wrote:
> >On Sat, 05 Jun 2010 00:16:42 +0200, Michael Snyder wrote:
> >>+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;

OK, it is a string after the question, not before it, my mistake.


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

During the gdb_test_no_output introduction there was not the "" -> ".*" change
in place.  Don't you plan to remove gdb_test_no_output now and put that
[string_to_regexp $command] into each gdb_test (gdb_test_multiple)?

That would solve even this case.

(For testing GDB asks at all there is "pendfunc1" before this test so dropping
this part of the test as you did is probably not a problem.)


Thanks,
Jan