Re: [commit] update_watchpoint is used with s/w watchpoints too

[Joel Brobecker <brobecker at adacore dot com>]

> There is still one part that I also find slightly confusing:
> 
>     If the watchpoint is disabled, do nothing
> 
> In fact, what this comments means is that, "if the disposition of the
> watchpoint is to be deleted at the next stop"...  I initially understood
> it as, if the watchpoint state was bp_disabled. Would it be OK if I
> changed the above to:
> 
>     If the watchpoint disposition is disp_del_at_next_stop, then do nothing.
> 
> ?

Any objection to the following patch? Will commit in a few days otherwise.

2008-12-29  Joel Brobecker  <brobecker@adacore.com>

        * breakpoint (update_watchpoint): Minor comment adjustment.

Thanks,
-- 
Joel

Attachment: breakpoint.c.diff
Description: Text document