avalon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Timothy Bennett <exterminat...@comcast.net>
Subject Re: Formatting
Date Mon, 22 Dec 2003 18:13:15 GMT
Niclas Hedhman wrote:
> Guys,
> 
> Without getting into a battle of "placement of braces", can we at least say 
> that;
> 
>         if (m_logger.isInfoEnabled()) m_logger.info("Servicing...");
> 
> is "undesirable", and it is Ok if I change that into two lines when I come 
> across it? (Too me it is incredible hard to read. Something in the pattern 
> recognition part of the brain that flips out... )
> 
> 
> Cheers
> Niclas

Is this my code, Niclas?  Feels familiar to me...

If so, strange as this may seem, I don't like one line if statements 
like this, so you certainly have my vote to make it two lines. AFAIC, 
put braces in there too.



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
For additional commands, e-mail: dev-help@avalon.apache.org


Mime
View raw message