Fixed bug where local variables to for-each clauses where flagged as …

…requiring the final modifier.
checkstyle · Dec 19, 2004 · 9bcd88c · 9bcd88c
1 parent c83b8ee
commit 9bcd88c
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/coding/FinalLocalVariableCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/coding/FinalLocalVariableCheck.java
@@ -112,7 +112,9 @@ public void visitToken(DetailAST aAST)
                 break;
             }
         case TokenTypes.VARIABLE_DEF:
-            if (aAST.getParent().getType() != TokenTypes.OBJBLOCK) {
+            if (aAST.getParent().getType() != TokenTypes.OBJBLOCK
+                && aAST.getParent().getType() != TokenTypes.FOR_EACH_CLAUSE)
+            {
                 insertVariable(aAST);
             }
             break;

diff --git a/src/testinputs/com/puppycrawl/tools/checkstyle/coding/InputFinalLocalVariable.java b/src/testinputs/com/puppycrawl/tools/checkstyle/coding/InputFinalLocalVariable.java
@@ -126,3 +126,14 @@ abstract class AbstractClass
 {
     public abstract void abstractMethod(int aParam);
 }
+
+class Blah
+{
+    static
+    {
+        for(int a : some.getInts())
+        {
+
+        }
+    }
+}