Re: [RFA] Add watchpoint test with both condition and thread keywords

[Pedro Alves <palves@redhat.com>]

On 03/08/2013 07:05 PM, Keith Seitz wrote:
> On 03/08/2013 11:00 AM, Keith Seitz wrote:
> 
>> /me gets more coffee
> 
> I should have done this BEFORE I sent the "revised" patch. :-(
> 
> Sorry about that.

Not a problem at all.

> testsuite/ChangeLog
> 2013-03-08  Keith Seitz  <keiths@redhat.com>
> 
>     * watchpoint.exp (test_no_hw_watchpoints: Add test using

Missing ).

>     both condition and thread.  Then delete the watchpoint.
> 
> diff --git a/gdb/testsuite/gdb.base/watchpoint.exp b/gdb/testsuite/gdb.base/watchpoint.exp
> index 5086ae5..08a4a9a 100644
> --- a/gdb/testsuite/gdb.base/watchpoint.exp
> +++ b/gdb/testsuite/gdb.base/watchpoint.exp
> @@ -808,6 +808,12 @@ proc test_no_hw_watchpoints {} {
>      "Watchpoint \[0-9\]*: ival3.*" \
>      "set slow conditional watch"
> 
> +    gdb_test "watch ival3 if  count > 1  thread 1 " \
> +         "Watchpoint \[0-9\]*: ival3.*" \
> +         "set slow condition watch w/thread"
> +
> +    gdb_test_no_output "delete \$bpnum" "delete watch w/cond and thread"

This now deletes the same breakpoint twice.  $bpnum is the number of
the last breakpoint/watchpoint created.

> +
>      gdb_test "continue" \
>      "Watchpoint \[0-9\]*: ival3.*Old value = 1.*New value = 2.*" \
>      "trigger slow conditional watch"


-- 
Pedro Alves