This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [RFA] scope_breakpoint -> internal breakpoint
- From: Keith Seitz <keiths at redhat dot com>
- To: gdb-patches at sources dot redhat dot com
- Date: Tue, 20 Aug 2002 13:06:16 -0700 (PDT)
- Subject: Re: [RFA] scope_breakpoint -> internal breakpoint
On Tue, 20 Aug 2002, Michael Snyder wrote:
> Good idea -- approved.
Wow, that was quick. Thanks! I've committed this patch.
Keith
> > ChangeLog
> > 2002-08-20 Keith Seitz <keiths@redhat.com>
> >
> > * breakpoints.c (watch_command_1): Use internal breakpoint
> > when setting a watchpoint_scope breakpoint.
> >
> > Patch
> > Index: breakpoint.c
> > ===================================================================
> > RCS file: /cvs/src/src/gdb/breakpoint.c,v
> > retrieving revision 1.80
> > diff -p -r1.80 breakpoint.c
> > *** breakpoint.c 17 Aug 2002 06:12:52 -0000 1.80
> > --- breakpoint.c 20 Aug 2002 18:19:42 -0000
> > *************** watch_command_1 (char *arg, int accessfl
> > *** 5331,5346 ****
> > if (prev_frame)
> > {
> > struct breakpoint *scope_breakpoint;
> > ! struct symtab_and_line scope_sal;
> > !
> > ! INIT_SAL (&scope_sal); /* initialize to zeroes */
> > ! scope_sal.pc = get_frame_pc (prev_frame);
> > ! scope_sal.section = find_pc_overlay (scope_sal.pc);
> > !
> > ! scope_breakpoint = set_raw_breakpoint (scope_sal,
> > ! bp_watchpoint_scope);
> > ! set_breakpoint_count (breakpoint_count + 1);
> > ! scope_breakpoint->number = breakpoint_count;
> >
> > scope_breakpoint->enable_state = bp_enabled;
> >
> > --- 5331,5338 ----
> > if (prev_frame)
> > {
> > struct breakpoint *scope_breakpoint;
> > ! scope_breakpoint = create_internal_breakpoint (get_frame_pc (prev_frame),
> > ! bp_watchpoint_scope);
> >
> > scope_breakpoint->enable_state = bp_enabled;
> >
>