QA·C Help - 3404

	3404	Statement contains a redundant * operator at top level. p++ means (p++) not (*p)++.

Redundancy

The dereference operation ( * ) in this expression is redundant because the value obtained is never used. Perhaps the intention was that the increment or decrement operation was to be applied after rather than before the dereference ? If so, the expression should be rewritten with parentheses as shown in the code example below.


/*PRQA S 489,506,2017,3199,3210,3205,3401,3403,3408,3440,3447,3673 ++*/

void foo(int *p)
{
    *p++;                         /* Message 3404 */

    (*p)++;                       /* OK */
}

See also:

3112 This statement has no side-effect - it can be removed.
3422 Statement contains a redundant operator at top level.
3423 Statement contains a redundant cast at top level.
3424 Statement contains a redundant & or | at top level.

Personality Groups | Glossary | Message Index

Contents