Re: Minutes of the GTK+ Team Meeting - 2010-09-21

2010/9/22 Emmanuele Bassi <ebassi gmail com>
Minutes of the GTK+ Team Meeting - 2010-09-21

• coding style conventions
- proposal: adopt the Clutter document, as it codifies the gtk+ as well
- investigate an indent script to distribute with gtk+

I know that these coding styles are already in action in a lot of gtk code.
But, i have to say that i don't see the point of :

"Curly braces should not be used for single statement blocks" :
- it's in my point of view a mistake to do this. It's error prone and hard to debug and know what was the intention of the developper here. And also, making 3 exceptions to this in the documentation is making it not easy to learn.
- In the opposite, making curly braces mandatory, removes all mistakes and bad interpretation of the code.

"In case of multiple statements, curly braces should be put on another indentation level"  and
"While curly braces for function definitions should rest on a new line, they should not add an indentation level ":
- this is a matter of taste of course. But i find it to be problematic for consistancy. Either do it for everything or not is a better idea. I think that curly braces should not be added in an indention level, only the embraced code.

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]