[LOGCXX-288] Logging statement requires braces around it in an if-else

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.10.0
• Fix Version/s: 0.11.0
• Environment: All
• Assignee: Curt Arnold
• Reporter: Dale King
• Created: 03/Jun/08 00:41
• Updated: 18/Jul/17 10:43
• Resolved: 17/Jun/08 21:02

Description

In converting some code to Log4CXX ended up with an if-else like this:

if(bMiniDumpSuccessful == TRUE)
LOG4CXX_FATAL( mainLogger, "Core Dump Successful!" );
else
...

This won't compile unless you either surround the logging statement with braces or leave off the semicolon. The above code essentially produces something like this:

if(bMiniDumpSuccessful == TRUE)
{
if (mainLogger->isFatalEnabled())

{ .... }

}
;
else

And it rightly complains that the else has no if associated with it.

Issue Links

• relates to: LOGCXX-490 conditional expression is constant