any g++ compilation warning to detect empty if statement

[lasing]

Hi all,

recently encounter following bugs:

if (a b); //<- bug in the careless ";" here
return 0;

Due to the extra ";" in the if statement line, it always return 0.

Anyone know g++ has any compiler - warning flag to detect this kind of
error? if not, is it any source code checking tools can do it ?

thanks,
Eric.

[Victor Bazarov]

lasing wrote:

recently encounter following bugs:

if (a b); //<- bug in the careless ";" here
return 0;

Due to the extra ";" in the if statement line, it always return 0.

Anyone know g++ has any compiler - warning flag to detect this kind of
error?

You need to ask in the compiler newsgroup about the settings for
any particular compiler. This is not a C++ language question.

if not, is it any source code checking tools can do it ?

I bet PC-lint can do it. [I just checked, it does: Info 721]
http://gimpel-online.com/MsgRef.html#721

V
--
Please remove capital 'A's when replying by e-mail
I do not respond to top-posted replies, please don't ask

[=?ISO-8859-1?Q?Erik_Wikstr=F6m?=]

On 2007-08-16 14:39, lasing wrote:

Hi all,

recently encounter following bugs:

if (a b); //<- bug in the careless ";" here
return 0;

Due to the extra ";" in the if statement line, it always return 0.

Anyone know g++ has any compiler - warning flag to detect this kind of
error? if not, is it any source code checking tools can do it ?

Not really what you asked for, but if your editor can indent the files
you work on, use that function, 'return 0;' will then be on the same
level as the if-statement which should make you suspicious.

--
Erik Wikström

[Frank Birbacher]

Hi!

lasing schrieb:

Anyone know g++ has any compiler - warning flag to detect this kind of
error? if not, is it any source code checking tools can do it ?

My gcc manual tells me it is covered by "-Wextra". But in practice my
g++ 4.1.2 does not complain about it -.-

Frank

[BobR]

lasing <yu******@gmail .comwrote in message...

Hi all,

recently encounter following bugs:

if (a b); file://<- bug in the careless ";" here
return 0;

Due to the extra ";" in the if statement line, it always return 0.

Anyone know g++ has any compiler - warning flag to detect this kind of
error? if not, is it any source code checking tools can do it ?
thanks, Eric.

Change your bad habits. If you get in the habit of always using
curly-braces, you will save yourself some of the 'gotchas'.

if( b ){}
if( a b ){ return 0; }
if( a b ){} else{ /* stuff */ }
for( /* .... */ ){}
for( /* .... */ ){ /* stuff */ }
while( a b ){ /* stuff */; ++b; }
while( /* something */ ){ ; } // if that's what you intended.
etc.

--
Bob R
POVrookie

[Victor Bazarov]

BobR wrote:

lasing <yu******@gmail .comwrote in message...

>Hi all,

recently encounter following bugs:

if (a b); file://<- bug in the careless ";" here
return 0;

Due to the extra ";" in the if statement line, it always return 0.

Anyone know g++ has any compiler - warning flag to detect this kind
of error? if not, is it any source code checking tools can do it ?
thanks, Eric.

Change your bad habits. If you get in the habit of always using
curly-braces, you will save yourself some of the 'gotchas'.

if( b ){}
if( a b ){ return 0; }
if( a b ){} else{ /* stuff */ }
for( /* .... */ ){}
for( /* .... */ ){ /* stuff */ }
while( a b ){ /* stuff */; ++b; }
while( /* something */ ){ ; } // if that's what you intended.
etc.

How does that help?

if (somefunkycondi tion);
{
/* plenty of stuff in curly braces */
}

V
--
Please remove capital 'A's when replying by e-mail
I do not respond to top-posted replies, please don't ask

[BobR]

Victor Bazarov <v.********@com Acast.netwrote in message...

>
How does that help?

if (somefunkycondi tion);
{
/* plenty of stuff in curly braces */
}

So sorry I tried to suggest something. I'll refrain from such in the future!

[ Beam me up Scottie! There is no intelligent life on Earth. ]
--
Bob R
POVrookie