[Glade-devel] Coding style
Paolo Borelli
pborelli@katamail.com
Fri, 17 Dec 2004 13:09:28 +0100
Il giorno ven, 17-12-2004 alle 11:34 +0000, Damon Chaplin ha scritto:
> On Thu, 2004-12-16 at 21:52, Tristan Van Berkom wrote:
> > On Thu, 16 Dec 2004 21:31:10 +0000, Archit Baweja
> > <bighead@users.sourceforge.net> wrote:
> > [...]
> > > Same for some while loops etc.
> > >
> > > I know its a little thing, but I'd recommend following one style, that
> > > too the old one. Keeping the coding style consistent is a good thing.
> >
> > Argh,
> > we went through all that a few months ago during the project
> > revival, the HACKING file has a note about that.
> >
> > The style is:
> > =========
> > if (...)
> > {
> > /* ... */
> > }
> > =========
> >
> > The other style (still present in some areas of the code) is
> > being removed slowly, patch by patch.
>
> I can't believe you did that. The old style is the one used throughout
> GNOME. There was absolutely no need to change it.
>
Just to clarify:
the preferred style was changed to "brace on a new line" back when
Joaquin was the maintainer: he started committing chuncks of code with
this style, so I asked him which one he preferred and went on converting
the files I was already editing, since I was fine with either styles and
the only thing I was concerned was consistency.
For what is worth large chunks of gnome use this style (8 spaces tab
indent, braces on a new line), for instance gedit and epiphany.
ciao
paolo