This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [RFC] add 'save-breakpoints' command
- From: Klee Dienes <klee at apple dot com>
- To: Michael Snyder <msnyder at cygnus dot com>
- Cc: gdb-patches at sources dot redhat dot com
- Date: 05 Dec 2001 00:09:36 -0500
- Subject: Re: [RFC] add 'save-breakpoints' command
- References: <E16BKC5-0002Wf-00@localhost> <3C0D520C.E9016E83@cygnus.com>
- Reply-to: Klee Dienes <klee at apple dot com>
> Could you elaborate a little bit on how this change works with
> shlib breakpoints, what the original_type field is for, the
> bp_tbreakpoint type, etc.? In what circumstances is the
> current breakpoint type different from the original breakpoint type?
The original_type field is for cases like 'tbreak' or 'thbreak' where
the bp_type gets set simply to 'breakpoint', but we nonethless want
the proper command to be written to the generated file. In these
cases we just set the original_type field to the appropriate value,
and use that when writing out the command to the file. It's also used
for non-used specified breakpoints --- in these cases the value is set
to 'bp_none', to indicate that it should not be written to the file.
> There are several mentions of future breaks; you should probably
> clean those up until you're ready to submit that feature.
Actually, upon further reflection, I should probably combine the two
diffs and submit them as one patch. Let me look into how well that
works out and post a new diff tomorrow.
> get_breakpoint_count is defined but never used.
Whoops. That's just a function needed by another of our patches that
somehow managed to sneak in. I'll remove it for now.