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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to