Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-09 Thread Manuel López-Ibáñez
On 9 May 2012 01:34, Cary Coutant wrote: >>> A suggestion: Make dwarf_name call the demangler, and then a (new?) a >>> function that converts a mangled decl to a human-readable string. In >>> any case, the pretty-printer does a lot of stuff that is mostly >>> useless for just printing a declaratio

Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-08 Thread Cary Coutant
>> A suggestion: Make dwarf_name call the demangler, and then a (new?) a >> function that converts a mangled decl to a human-readable string. In >> any case, the pretty-printer does a lot of stuff that is mostly >> useless for just printing a declaration (translation, wrapping, etc.). >> >> Bonus p

Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-08 Thread Gabriel Dos Reis
On Tue, May 8, 2012 at 4:38 PM, Manuel López-Ibáñez wrote: > A suggestion: Make dwarf_name call the demangler, and then a (new?) a > function that converts a mangled decl to a human-readable string. In > any case, the pretty-printer does a lot of stuff that is mostly > useless for just printing a

Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-08 Thread Manuel López-Ibáñez
On 8 May 2012 18:20, Sterling Augustine wrote: > On Mon, May 7, 2012 at 6:44 PM, Gabriel Dos Reis > wrote: >> On Mon, May 7, 2012 at 7:10 PM, Sterling Augustine >> wrote: >>> This is the second in the series of patches to make c decl pretty printing >>> more closely match the demangler. A full e

Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-08 Thread Gabriel Dos Reis
On Tue, May 8, 2012 at 11:20 AM, Sterling Augustine wrote: > Do you have a suggestion for fixing the disagreement? I would love to > add this as a parameter somewhere, but the decision is very deep in > the internals of the pretty printer. I disagree with your characterization that the pretty-pr

Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-08 Thread Sterling Augustine
On Mon, May 7, 2012 at 6:44 PM, Gabriel Dos Reis wrote: > On Mon, May 7, 2012 at 7:10 PM, Sterling Augustine > wrote: >> This is the second in the series of patches to make c decl pretty printing >> more closely match the demangler. A full explanation is here: >> >> http://gcc.gnu.org/ml/gcc-patc

Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-07 Thread Gabriel Dos Reis
On Mon, May 7, 2012 at 7:10 PM, Sterling Augustine wrote: > This is the second in the series of patches to make c decl pretty printing > more closely match the demangler. A full explanation is here: > > http://gcc.gnu.org/ml/gcc-patches/2012-05/msg00512.html > > OK for mainline? Now I realize som

[C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-07 Thread Sterling Augustine
This is the second in the series of patches to make c decl pretty printing more closely match the demangler. A full explanation is here: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg00512.html OK for mainline? Sterling 2012-05-07 Sterling Augustine * c-family/c-pretty-print.c (pp_c