[previous] 2980 [next] The value of this function parameter is never used before being modified.
Redundancy

The value passed in this function parameter is always modified before being used and so the parameter is effectively redundant. Is this really intended ?

For example: 


/*PRQA S 2017,2986,3195,3203,3227,3408 ++*/

int foo(int n, int x)    /* 2980 */
{
    n = x + 1;

    return n;
}

QA·C Source Code Analyser 8.1.2
© 2013 Programming Research.
www.programmingresearch.com 		Personality Groups | Glossary | Message Index Contents