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

Michael Snyder msnyder@vmware.com
Mon Jun 7 23:00:00 GMT 2010


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



More information about the Gdb-patches mailing list