V639. Consider inspecting the expression for function call. It is possible that one of the closing ')' parentheses was positioned incorrectly.


The analyzer has detected a potential error: a suspicious function call is present which is followed by commas and expressions. Perhaps these expressions should be part of the function call.

This error usually occurs if a function is called inside a conditional operator and the function has arguments by default. In this case you may easily make a mistake writing a closing parenthesis in a wrong place. What is dangerous about these errors is that the code is compiled and executed without errors. Consider the following sample of incorrect code:

bool rTuple(int a, bool Error = true);
....
if (rTuple(exp), false)
{
 ....
}

The closing parenthesis put in a wrong place will cause two errors at once:

1) The 'Error' argument will equal 'true' when calling the 'rTuple' function, though the programmer meant it to be 'false'.

2) The comma operator ',' returns the value of the right part. It means that the (rTuple(exp), false) condition will always be 'false'

This is the fixed code:

if (rTuple(exp, false))
{
 ....
}

According to Common Weakness Enumeration, potential errors found by using this diagnostic are classified as CWE-685.

You can look at examples of errors detected by the V639 diagnostic.


Bugs Found

Checked Projects
334
Collected Errors
12 668