In some header file that I cannot change, I have the following set of definitions:
#define FLAG1 (0x0000_0001)
#define FLAG2 (0x0000_0002)
...
Then in my code I use them in switch:
switch (aaa) {
case FLAG1:
....
case FLAG2:
....
}
As a result, Coverity reports two defects per case label:
RW.EXP_RPAREN:
Event exp_rparen: expected a ")"
RW.CASE_LABEL_CONFLICT:
Event case_label_conflict: case label value has already appeared in
this switch at line XX
What is wrong with these shortcuts? Does it violate C standards?
source
share