[Freeipa-devel] [PATCH] initial commit of log watcher (lwatch)

[Simo Sorce]

On Fri, 2009-07-24 at 11:53 -0400, John Dennis wrote:
> Our coding standard recommends no braces when there is only a single 
> expression. However I strongly disagree with this, in fact many
> coding 
> standard recommend exactly the opposite, to always use braces! Why? 
> Because I've seen plenty examples of bugs introduced when a
> programmer 
> added a new expression to the conditional and totally broke the
> program 
> logic. Usually these mistakes aren't caught until much later when
> things 
> mysterously fail. I consider the use of braces a simple and effective 
> example of defensive programming.
> 

Please propose to amend the coding style in a separate mail.
I have had mixed feelings about this, but I think I might agree with you
on this point, so maybe we will get enough support to change the coding
style on this point.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York