This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PATCH comment in infrun.c


On Tue, May 03, 2005 at 09:41:49AM +1000, Ben Elliston wrote:
> > It needs updating, not removal; there's still a reference in remote.c.
> 
> Better?

OK, thanks.

> 
> Index: infrun.c
> ===================================================================
> RCS file: /home/bje/src-cvs/src/gdb/infrun.c,v
> retrieving revision 1.199
> diff -u -p -r1.199 infrun.c
> --- infrun.c    24 Feb 2005 13:51:32 -0000      1.199
> +++ infrun.c    2 May 2005 23:41:23 -0000
> @@ -1233,11 +1233,10 @@ int stepped_after_stopped_by_watchpoint;
>  void
>  handle_inferior_event (struct execution_control_state *ecs)
>  {
> -  /* NOTE: cagney/2003-03-28: If you're looking at this code and
> -     thinking that the variable stepped_after_stopped_by_watchpoint
> -     isn't used, then you're wrong!  The macro STOPPED_BY_WATCHPOINT,
> -     defined in the file "config/pa/nm-hppah.h", accesses the variable
> -     indirectly.  Mutter something rude about the HP merge.  */
> +  /* NOTE: bje/2005-05-02: If you're looking at this code and thinking
> +     that the variable stepped_after_stopped_by_watchpoint isn't used,
> +     then you're wrong!  See remote.c:remote_stopped_data_address.  */
> +
>    int sw_single_step_trap_p = 0;
>    int stopped_by_watchpoint = -1;      /* Mark as unknown.  */
> 



-- 
Daniel Jacobowitz
CodeSourcery, LLC


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]