This is the mail archive of the gdb-patches@sourceware.org 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: [RFA] fix disassemble foo::bar::~bar


>>>>> "Doug" == Doug Evans <dje@google.com> writes:

Doug> I happened to try this and noticed it failing with
Doug> "name of destructor must equal name of class".

Doug> This is because destructor_name_p doesn't handle foo::bar
Doug> for the type's name.

Thanks for finding this.

Doug> +      char *cp = strchr (type_name, '<');

I wonder whether this kind of parsing is sufficient.
It seems like there could be confounding cases.

I wonder what would happen if we just removed destructor_name_p,
or alternatively made it work by examining the type's function fields,
looking for a match.

If you've considered the various error cases in your analysis, then I
have no problem with your patch.

Tom


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