Re: gengtype indentation issues

2010-09-24 Thread Ian Lance Taylor
Paolo Bonzini writes: > On 09/23/2010 08:49 PM, Ian Lance Taylor wrote: >> Diego Novillo writes: >> >>> I'd suggest sending one initial patch fixing indentation issues and >>> then sending your functional changes on top of the first patch. >> >> Yes. >> >> Basile, I'm sorry you have to struggle

Re: gengtype indentation issues

2010-09-24 Thread Paolo Bonzini
On 09/24/2010 11:29 AM, Basile Starynkevitch wrote: I don't understand what concrete steps are you suggesting. As explained above, I am using M-x indent-region but it does indent gengtype code because gengtype is mis-indented from the beginning. Maybe my patches will never go into trunk because

Re: gengtype indentation issues

2010-09-24 Thread Basile Starynkevitch
On Fri, Sep 24, 2010 at 10:51:43AM +0200, Paolo Bonzini wrote: > On 09/23/2010 08:49 PM, Ian Lance Taylor wrote: > >Diego Novillo writes: > > > >>I'd suggest sending one initial patch fixing indentation issues and > >>then sending your functional changes on top of the first patch. > > > >Yes. I d

Re: gengtype indentation issues

2010-09-24 Thread Paolo Bonzini
On 09/23/2010 08:49 PM, Ian Lance Taylor wrote: Diego Novillo writes: I'd suggest sending one initial patch fixing indentation issues and then sending your functional changes on top of the first patch. Yes. Basile, I'm sorry you have to struggle with some poor existing indentation in gengty

Re: gengtype indentation issues

2010-09-23 Thread Ian Lance Taylor
Diego Novillo writes: > I'd suggest sending one initial patch fixing indentation issues and > then sending your functional changes on top of the first patch. Yes. Basile, I'm sorry you have to struggle with some poor existing indentation in gengtype.c, but the GNU style indentation is not all t

Re: gengtype indentation issues

2010-09-23 Thread Diego Novillo
I'd suggest sending one initial patch fixing indentation issues and then sending your functional changes on top of the first patch. Diego.

gengtype indentation issues

2010-09-23 Thread Basile Starynkevitch
Hello All, References http://gcc.gnu.org/ml/gcc-patches/2010-09/msg01739.html http://gcc.gnu.org/ml/gcc-patches/2010-09/msg01740.html http://gcc.gnu.org/ml/gcc-patches/2010-09/msg01742.html http://gcc.gnu.org/ml/gcc-patches/2010-09/msg01743.html http://gcc.gnu.org/ml/gcc-patches/2010-09/