[Glade-devel] Coding style

Andrew J. Montalenti pixelmonkey@pixelmonkey.org
Sat, 18 Dec 2004 17:22:51 -0500


Hey!

On Sat, 2004-12-18 at 10:24 +0000, Damon Chaplin wrote:
> On Fri, 2004-12-17 at 12:09, Paolo Borelli wrote:
> 
> > 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.
> 
> OK. That is fair enough. Though I would have preferred it left as the
> standard GNOME style.

GNOME guidelines pretty fairly state that patches should be written
using whatever style the author writes the code in.  For community
projects like Glade, a style should be _decided upon_ (and I believe
Tristan and co. have decided on new lines for braces) and then any
patches should follow this style.  Style is such a trivial issue,
anyway!

Most of us are pretty good programmers, let's not quarrel over stuff
like this.  Write a short sed script that converts your style into Glade
style, if you like, and apply it before submitting patches :-)

I personally prefer braces on the same line for control blocks (if, for,
while) and braces on new lines for function definitions, but hey, that's
me.  If I write a project from scratch ever, that'll be my style.  In
the end, when submitting patches I convert to the style of the author,
which is what everyone should do, probably ;-)

-Andrew