fix code

Author: nidi3

code-assert/src/main/java/guru/nidi/codeassert/pmd/CpdMatchCollector.java

@@ -19,7 +19,6 @@
 import net.sourceforge.pmd.cpd.Mark;
 import net.sourceforge.pmd.cpd.Match;
 
-import java.util.Iterator;
 import java.util.List;
 
 import static guru.nidi.codeassert.util.ListUtils.andJoin;
@@ -44,7 +43,7 @@ public String toString() {
     @Override
     protected ActionResult doAccept(Match issue, Ignore action) {
         ActionResult res = ActionResult.undecided(null);
-        for (Mark mark : issue) {
+        for (final Mark mark : issue) {
             final Language language = Language.byFilename(mark.getFilename());
             res = res.orMoreQuality(action.accept(
                     new NamedLocation(mark.getSourceCodeSlice(), language, PmdUtils.className(mark), "", false)));

code-assert/src/main/java/guru/nidi/codeassert/pmd/CpdMatcher.java

@@ -51,7 +51,7 @@ private String printMatch(Match match) {
 
     private List<Mark> getMarks(Match match) {
         final List<Mark> marks = new ArrayList<>();
-        for (Mark mark : match) {
+        for (final Mark mark : match) {
             marks.add(mark);
         }
         marks.sort(MARK_COMPARATOR);

code-assert/src/test/java/guru/nidi/codeassert/checkstyle/CheckstyleTest.java

@@ -73,8 +73,8 @@ void google() {
                         + line(WARNING, "abbreviation.as.word", MAIN, "pmd/PmdRulesets", 165, "Abbreviation in name 'serialVersionUID' must contain no more than '1' capital letters.")
                         + line(WARNING, "abbreviation.as.word", MAIN, "pmd/PmdRulesets", 210, "Abbreviation in name 'serialVersionUID' must contain no more than '1' capital letters.")
                         + line(WARNING, "empty.line.separator", TEST, "io/ModelVisualizerTest", 59, "',' should be separated from previous statement.")
-                        + line(WARNING, "indentation.error", MAIN, "model/InternalTypeInPublicApiMatcher", 48, "'lambda arguments' have incorrect indentation level 32, expected level should be 28.")
-                        + line(WARNING, "indentation.error", MAIN, "model/InternalTypeInPublicApiMatcher", 50, "'lambda arguments' have incorrect indentation level 32, expected level should be 28.")
+                        + line(WARNING, "indentation.error", MAIN, "model/InternalTypeInPublicApiMatcher", 45, "'lambda arguments' have incorrect indentation level 32, expected level should be 28.")
+                        + line(WARNING, "indentation.error", MAIN, "model/InternalTypeInPublicApiMatcher", 47, "'lambda arguments' have incorrect indentation level 32, expected level should be 28.")
                         + line(WARNING, "one.top.level.class", TEST, "model/ExampleConcreteClass", 79, "Top-level class ExamplePackageClass has to reside in its own source file.")
                         + line(WARNING, "overload.methods.declaration", MAIN, "config/BaseCollector", 58, "Overload methods should not be split. Previous overloaded method located at line '52'.")
                         + line(WARNING, "overload.methods.declaration", MAIN, "config/BaseCollector", 69, "Overload methods should not be split. Previous overloaded method located at line '56'.")