[obvious?] Revert change to gdb.cp/ovldbreak.exp
Commit Message
This reverts the timeout handling (removed by
018572b88885ae67d22612937fa1e4fd98d5f5ad) for gdb.cp/ovldbreak.exp until we
decide what to do about this particular function.
I take it it is obvious?
gdb/testsuite/ChangeLog:
2016-12-02 Luis Machado <lgustavo@codesourcery.com>
* gdb.base/ovldbreak.exp (take_gdb_out_of_choice_menu): Restore
timeout handling.
---
gdb/testsuite/gdb.cp/ovldbreak.exp | 4 ++++
1 file changed, 4 insertions(+)
Comments
On 12/02/2016 05:13 PM, Luis Machado wrote:
> This reverts the timeout handling (removed by
> 018572b88885ae67d22612937fa1e4fd98d5f5ad) for gdb.cp/ovldbreak.exp until we
> decide what to do about this particular function.
>
> I take it it is obvious?
>
> gdb/testsuite/ChangeLog:
>
> 2016-12-02 Luis Machado <lgustavo@codesourcery.com>
>
> * gdb.base/ovldbreak.exp (take_gdb_out_of_choice_menu): Restore
> timeout handling.
OK.
Thanks,
Pedro Alves
On 12/02/2016 11:32 AM, Pedro Alves wrote:
> On 12/02/2016 05:13 PM, Luis Machado wrote:
>> This reverts the timeout handling (removed by
>> 018572b88885ae67d22612937fa1e4fd98d5f5ad) for gdb.cp/ovldbreak.exp until we
>> decide what to do about this particular function.
>>
>> I take it it is obvious?
>>
>> gdb/testsuite/ChangeLog:
>>
>> 2016-12-02 Luis Machado <lgustavo@codesourcery.com>
>>
>> * gdb.base/ovldbreak.exp (take_gdb_out_of_choice_menu): Restore
>> timeout handling.
>
> OK.
>
> Thanks,
> Pedro Alves
>
Thanks, pushed as 008459ea88ad4fdcb7da37dad7f68888ef6ecc47.
@@ -58,6 +58,10 @@ proc take_gdb_out_of_choice_menu {} {
gdb_test_multiple " " " " {
-re ".*$gdb_prompt $" {
}
+ timeout {
+ perror "could not resynchronize to command prompt (timeout)"
+ continue
+ }
}
}