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 ccout...@google.com 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 Sterling Augustine
On Mon, May 7, 2012 at 6:44 PM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Mon, May 7, 2012 at 7:10 PM, Sterling Augustine saugust...@google.com wrote: This is the second in the series of patches to make c decl pretty printing more closely match the demangler. A full explanation

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 saugust...@google.com 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

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 saugust...@google.com wrote: On Mon, May 7, 2012 at 6:44 PM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Mon, May 7, 2012 at 7:10 PM, Sterling Augustine saugust...@google.com wrote: This is the second in the series of patches to make c decl

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 lopeziba...@gmail.com 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

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 point if all

[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 saugust...@google.com *

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 saugust...@google.com 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?