Re: GTK+ policy (was RE:rendering-cleanup-next)

2010-09-14 Thread Jon Cruz
On Sep 13, 2010, at 10:49 PM, Kristian Rietveld wrote: To add to this, it does not specify how to deal with braces in nested if-statements. When I start to nest if-statements (also with a single-statement body) I typically start adding braces, because it is a bit clearer and avoids the

Re: GTK+ policy (was RE:rendering-cleanup-next)

2010-09-14 Thread Tomeu Vizoso
On Tue, Sep 14, 2010 at 09:14, Jon Cruz j...@joncruz.org wrote: On Sep 13, 2010, at 10:49 PM, Kristian Rietveld wrote: To add to this, it does not specify how to deal with braces in nested if-statements.  When I start to nest if-statements (also with a single-statement body) I typically

Re: GTK+ policy (was RE:rendering-cleanup-next)

2010-09-14 Thread Andrés G. Aragoneses
El 14/09/10 09:46, Tomeu Vizoso escribió: On Tue, Sep 14, 2010 at 09:14, Jon Cruz j...@joncruz.org wrote: On Sep 13, 2010, at 10:49 PM, Kristian Rietveld wrote: To add to this, it does not specify how to deal with braces in nested if-statements. When I start to nest if-statements (also

Re: GTK+ policy (was RE:rendering-cleanup-next)

2010-09-14 Thread Owen Taylor
On Mon, 2010-09-13 at 21:48 -0400, Paul Davis wrote: On Mon, Sep 13, 2010 at 6:27 PM, Matthias Clasen matthias.cla...@gmail.com wrote: 2010/9/13 Thomas Wood t...@gnome.org: Clutter's (very detailed) coding style document may be useful here, since it has a very similar coding style to

Re: GTK+ policy (was RE:rendering-cleanup-next)

2010-09-14 Thread Paul Davis
On Tue, Sep 14, 2010 at 9:48 AM, Owen Taylor otay...@redhat.com wrote: But remember, GTK+ is using GNU style bracing and that pretty much takes the question out of the matter. Doing:  if (conditition)   {     single_statement ();   }  else   {     another_single_statement ();   }

Re: GTK+ policy (was RE:rendering-cleanup-next)

2010-09-13 Thread Thomas Wood
On Sun, 2010-09-12 at 18:50 +0200, Javier Jardón wrote: 2010/9/12 Matthias Clasen matthias.cla...@gmail.com: We don't have a written-down policy, beyond 'fit in locally'. But I have become increasingly annoyed by trailing whitespace and mixed-in tabs, since they do show up in my editor

Re: GTK+ policy (was RE:rendering-cleanup-next)

2010-09-13 Thread Matthias Clasen
2010/9/13 Thomas Wood t...@gnome.org: Clutter's (very detailed) coding style document may be useful here, since it has a very similar coding style to GTK+: http://git.clutter-project.org/clutter/tree/doc/CODING_STYLE Yes, I think we could basically adopt this word-by-word.

Re: GTK+ policy (was RE:rendering-cleanup-next)

2010-09-13 Thread Paul Davis
On Mon, Sep 13, 2010 at 6:27 PM, Matthias Clasen matthias.cla...@gmail.com wrote: 2010/9/13 Thomas Wood t...@gnome.org: Clutter's (very detailed) coding style document may be useful here, since it has a very similar coding style to GTK+:

Re: GTK+ policy (was RE:rendering-cleanup-next)

2010-09-13 Thread Kristian Rietveld
On Sep 14, 2010, at 3:48 AM, Paul Davis wrote: On Mon, Sep 13, 2010 at 6:27 PM, Matthias Clasen matthias.cla...@gmail.com wrote: 2010/9/13 Thomas Wood t...@gnome.org: Clutter's (very detailed) coding style document may be useful here, since it has a very similar coding style to GTK+:

GTK+ policy (was RE:rendering-cleanup-next)

2010-09-12 Thread Javier Jardón
2010/9/12 Matthias Clasen matthias.cla...@gmail.com: We don't have a written-down policy, beyond 'fit in locally'. But I have become increasingly annoyed by trailing whitespace and mixed-in tabs, since they do show up in my editor nowadays. So maybe we should agree on a policy and put it in