Re: Regression for -gstabs+ [Re: FYI: fix spelling of "delete" in c-exp.y]

Tom Tromey
2012-06-19T14:10:10Z

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