This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: add set cp-abi command
- From: Daniel Jacobowitz <drow at mvista dot com>
- To: Jim Ingham <jingham at apple dot com>
- Cc: gdb-patches at sources dot redhat dot com
- Date: Tue, 9 Apr 2002 20:54:02 -0400
- Subject: Re: add set cp-abi command
- References: <20020322140939.A26911@nevyn.them.org> <B80D7663-4C19-11D6-B08E-000393540DDC@apple.com>
On Tue, Apr 09, 2002 at 05:27:19PM -0700, Jim Ingham wrote:
> I got distracted from this for a little while... Here is a final
> version with something in the texi file as well.
>
> ChangeLog:
>
> 2002-03-12 James Ingham <jingham@apple.com>
>
> * cp-abi.c (set_cp_abi_cmd, show_cp_abi_cmd,): New functions,
> allow you to set & show cplus abi's in case gdb gets it wrong.
> (set_cp_abi_as_auto_default): New function, set the "auto" abi
> to be this abi.
> (is_cp_abi_auto_p): New function, say whether the current abi
> is the default or not.
> (_initialize_cp_abi): Define the cp-abi switching commands.
> * cp-abi.h: declare the new functions.
> * minsyms.c (install_minimal_symbols): don't switch the cp_abi
> unless the current abi is auto.
> * gnu-v2-abi.c (_initialize_gnu_v2_abi): don't switch to gnu-v2,
> but set it as the auto_default instead.
>
> Okay to check in?
I think we may have misplaced a revision here...
! internal_error (__FILE__, __LINE__,
! "Too many CP ABIs, please increase CP_ABI_MAX in cp-abi.c");
C++ ABIs.
! char *longname; /* These two can't be const, because I need to */
! char *doc; /* change the name for the auto abi. */
GNU comment formatting. Something like:
/* These two are not constant because they are changed at runtime
for the auto ABI. */
char *longname;
char *doc;
More importantly:
! if (cp_abi_is_auto_p ())
! set_cp_abi_as_auto_default ("gnu-v3");
Shouldn't you lose the cp_abi_is_auto_p () check there?
If you agree with all three, feel free to update them and check it in.
--
Daniel Jacobowitz Carnegie Mellon University
MontaVista Software Debian GNU/Linux Developer