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: Regression for -gstabs+ [Re: FYI: fix spelling of "delete" in c-exp.y]


Jan> this exactly reverts my change:
Jan> 	[WIP patch] Fix crash regressions after libiberty/ update
Jan> 	http://sourceware.org/ml/gdb-patches/2012-01/msg00261.html

Jan> I made there this change to match libiberty/cp-demangle.c
Jan> cplus_demangle_operators[].

Sorry.
I'll revert it and figure out a different fix.

Why are we spending any time at all on stabs and C++?  IIRC that never
really worked that well, and stabs are past obsolete.

>> Debugging one of these showed that there were some extra spaces in
>> c-exp.y, that caused problems in this test case.  This patch removes the
>> spaces.

Jan> "Extra" why?  There seem to be missing spaces vs. the demanger now.

I will debug it again, but right now I don't understand this.
The demangler will add argument types as well.
When can you end up with a symbol name with a trailing space?

Tom


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