37 lines
808 B
Plaintext
37 lines
808 B
Plaintext
|
== Why is this an issue?
|
||
|
|
|
||
|
|
include::../description.adoc[]
|
||
|
|
|
||
|
|
=== Code examples
|
||
|
|
|
||
|
|
==== Noncompliant code example
|
||
|
|
|
||
|
|
[source,cpp,diff-id=1,diff-type=noncompliant]
|
||
|
|
----
|
||
|
|
void doSomething() {
|
||
|
|
; // Noncompliant - was used as a kind of TODO marker
|
||
|
|
}
|
||
|
|
|
||
|
|
if (complicated.expression.foo()); // Noncompliant - the condition doesn't make sense
|
||
|
|
bar();
|
||
|
|
----
|
||
|
|
|
||
|
|
==== Compliant solution
|
||
|
|
|
||
|
|
[source,cpp,diff-id=1,diff-type=compliant]
|
||
|
|
----
|
||
|
|
void doSomething() {
|
||
|
|
}
|
||
|
|
|
||
|
|
if (complicated.expression.foo()) {
|
||
|
|
bar();
|
||
|
|
}
|
||
|
|
----
|
||
|
|
|
||
|
|
== Resources
|
||
|
|
|
||
|
|
=== Documentation
|
||
|
|
|
||
|
|
* https://wiki.sei.cmu.edu/confluence/x/5dUxBQ[CERT, MSC12-C.] - Detect and remove code that has no effect or is never executed
|
||
|
|
* https://wiki.sei.cmu.edu/confluence/x/WtYxBQ[CERT, EXP15-C.] - Do not place a semicolon on the same line as an if, for, or while statement
|