[previous] 2985 [next] This operation is redundant. The value of the result is always that of the left-hand operand.
Redundancy

This operation appears to be redundant because the left-hand operand can always be used instead.

For example:


/*PRQA S 2017,2983,3198,3203,3408,3447 ++*/

void foo(unsigned int i, unsigned int j)
{
    if (j > i)
    {
        i % j;         /* 2985 */
    }

    i << 0;            /* 2985 */

    j | 0;             /* 2985 */

    j & 0xFFFFFFFFu;   /* 2985 */
}

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