From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Elizabeth Chastain <chastain@cygnus.com>
To: dberlin@redhat.com, gdb-patches@sources.redhat.com
Subject: Re: [PATCH] Start abstraction of C++ abi's
Date: Sun, 18 Feb 2001 16:09:00 -0000
Message-id: <200102190009.QAA20605@bosch.cygnus.com>
X-SW-Source: 2001-02/msg00349.html

Looks like a good idea to me.  "set demangle" currently shows five
different styles (gnu, lucid, arm, hp, edg).  We know that gnu encompasses
two different manglers by itself.  We gotta move these things into
virtual functions.

I won't comment on the directory structure (cp/cp-abi versus cp-abi);
maybe a maintainer would like to think about that.

Some specific comments:

> ! 		   constructor_prefix_p (physname) 
> ! 		   || destructor_prefix_p (physname)
> ! 		   || STREQN (method_name, "~", 1);

Does that lingering STREQN have to be there?

>   		  demangled_name =
>   		    cplus_demangle (mangled_name,
> ! 				    DMGL_ANSI | DMGL_PARAMS | DMGL_VERBOSE);

This looks like you mixed in a bit from a different patch -- there is no
DMGL_VERBOSE in the gdb cvs tree yet.

Beyond that I haven't proof-read the code yet.

Michael