[asterisk-dev] Coding guidelines change (proposed)

Tzafrir Cohen tzafrir.cohen at xorcom.com
Tue Feb 26 14:30:02 CST 2008


On Tue, Feb 26, 2008 at 11:46:37AM -0600, Tilghman Lesher wrote:
> I'd like to propose a change to the coding guidelines, and I'm airing it here
> for specific feedback before we make this change.  Currently, our standard
> is that if you have a single statement following an if/for/while construct,
> then we do not require parentheses around the single statement.  While this
> is legal C, it has caused a number of issues in the past, where another
> statement was added at that indentation level, but without adding the
> necessary braces.
> 
> I'd like to change the coding guidelines that the braces are no longer
> optional; that they are mandatory, and if this change is approved, a janitor
> project will be posted shortly to convert legacy code to the new standard.

Sounds like a good idea. How good is indent in that task?

> 
> Are there any comments or objections before we implement this change?

Could you please point to one or two such cases?

Current coding guidelines already require that a nested block will have
braces. I wonder if those issues are related to bad indentation.

-- 
               Tzafrir Cohen
icq#16849755              jabber:tzafrir.cohen at xorcom.com
+972-50-7952406           mailto:tzafrir.cohen at xorcom.com
http://www.xorcom.com  iax:guest at local.xorcom.com/tzafrir



More information about the asterisk-dev mailing list