Re: [PATCH 1/2] make the c++ pretty printer inherit from the C one instead of include it

2013-08-04 Thread Gabriel Dos Reis
On Sun, Aug 4, 2013 at 8:37 PM, Trevor Saunders wrote: > On Wed, Jul 31, 2013 at 10:02:29PM -0500, Gabriel Dos Reis wrote: >> * declare the "pointer to function fields" as virtual functions -- >> that is what I meant >> with the (necessarily poor) emulation through the casts. > > I guess you

Re: [PATCH 1/2] make the c++ pretty printer inherit from the C one instead of include it

2013-08-04 Thread Trevor Saunders
On Wed, Jul 31, 2013 at 10:02:29PM -0500, Gabriel Dos Reis wrote: > * declare the "pointer to function fields" as virtual functions -- > that is what I meant > with the (necessarily poor) emulation through the casts. I guess you'll work on this later in the patch series you're sending, but i

Re: [PATCH 1/2] make the c++ pretty printer inherit from the C one instead of include it

2013-07-31 Thread Gabriel Dos Reis
On Wed, Jul 31, 2013 at 10:16 PM, Trevor Saunders wrote: > On Wed, Jul 31, 2013 at 10:02:29PM -0500, Gabriel Dos Reis wrote: >> On Wed, Jul 31, 2013 at 8:14 PM, Trevor Saunders >> wrote: >> > bootstrapped and same test results on x86_64-linux-gnu against r201084 >> > >> > gcc/cp/ >> > *

Re: [PATCH 1/2] make the c++ pretty printer inherit from the C one instead of include it

2013-07-31 Thread Trevor Saunders
On Wed, Jul 31, 2013 at 10:02:29PM -0500, Gabriel Dos Reis wrote: > On Wed, Jul 31, 2013 at 8:14 PM, Trevor Saunders > wrote: > > bootstrapped and same test results on x86_64-linux-gnu against r201084 > > > > gcc/cp/ > > * cxx-pretty-print.h (cxx_pretty_printer): inherit from > > c_prett

Re: [PATCH 1/2] make the c++ pretty printer inherit from the C one instead of include it

2013-07-31 Thread Gabriel Dos Reis
On Wed, Jul 31, 2013 at 8:14 PM, Trevor Saunders wrote: > bootstrapped and same test results on x86_64-linux-gnu against r201084 > > gcc/cp/ > * cxx-pretty-print.h (cxx_pretty_printer): inherit from > c_pretty_printer > instead of include it. > * (cxx_pretty_print.c): adju

[PATCH 1/2] make the c++ pretty printer inherit from the C one instead of include it

2013-07-31 Thread Trevor Saunders
bootstrapped and same test results on x86_64-linux-gnu against r201084 gcc/cp/ * cxx-pretty-print.h (cxx_pretty_printer): inherit from c_pretty_printer instead of include it. * (cxx_pretty_print.c): adjust accordingly. --- gcc/cp/cxx-pretty-print.c | 40 ++