diff --git a/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/metrics/impl/CycloMetric.java b/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/metrics/impl/CycloMetric.java
index 260849b1f..63db78c19 100644
--- a/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/metrics/impl/CycloMetric.java
+++ b/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/metrics/impl/CycloMetric.java
@@ -13,7 +13,6 @@
import net.sourceforge.pmd.lang.java.ast.ASTConditionalOrExpression;
import net.sourceforge.pmd.lang.java.ast.ASTExpression;
import net.sourceforge.pmd.lang.java.ast.ASTMethodOrConstructorDeclaration;
-import net.sourceforge.pmd.lang.java.ast.JavaNode;
import net.sourceforge.pmd.lang.java.ast.JavaParserVisitor;
import net.sourceforge.pmd.lang.java.metrics.JavaMetrics;
import net.sourceforge.pmd.lang.java.metrics.api.JavaOperationMetricKey;
@@ -30,21 +29,26 @@
*
*
The standard version of the metric complies with McCabe's original definition [3]:
*
- *
- +1 for every control flow statement ({@code if, case, catch, throw, do, while, for, break, continue}) and
+ *
+ * - +1 for every control flow statement ({@code if, case, catch, throw, do, while, for, break, continue}) and
* conditional expression ({@code ? : }). Notice switch cases count as one, but not the switch itself: the point is that
- * a switch should have the same complexity value as the equivalent series of {@code if} statements.
- {@code else},
- * {@code finally} and {@code default} don't count;
- +1 for every boolean operator ({@code &&, ||}) in the guard
- * condition of a control flow statement. That's because Java has short-circuit evaluation semantics for boolean
- * operators, which makes every boolean operator kind of a control flow statement in itself.
+ * a switch should have the same complexity value as the equivalent series of {@code if} statements.
+ * - {@code else}, {@code finally} and {@code default} don't count;
+ *
- +1 for every boolean operator ({@code &&, ||}) in the guard condition of a control flow statement. That's because
+ * Java has short-circuit evaluation semantics for boolean operators, which makes every boolean operator kind of a
+ * control flow statement in itself.
+ *
*
* Version {@link CycloVersion#IGNORE_BOOLEAN_PATHS}: Boolean operators are not counted, which means that empty
* fall-through cases in {@code switch} statements are not counted as well.
*
*
References:
*
- *
+ *
+ * - [1] Lanza, Object-Oriented Metrics in Practice, 2005.
+ *
- [2] McCabe, A Complexity Measure, in Proceedings of the 2nd ICSE (1976).
+ *
- [3] Sonarqube online documentation
+ *
*
* @author Clément Fournier
* @since June 2017