Revert "Fix no new variable inspection for shadowing in 'for clause' case"

This reverts commit b8eb9d5.

Author: zolotov

src/com/goide/inspections/GoNoNewVariablesInspection.java

@@ -16,7 +16,6 @@
 
 package com.goide.inspections;
 
-import com.goide.psi.GoForClause;
 import com.goide.psi.GoShortVarDeclaration;
 import com.goide.psi.GoVarDefinition;
 import com.goide.psi.GoVisitor;
@@ -25,7 +24,6 @@
 import com.intellij.openapi.util.TextRange;
 import com.intellij.psi.PsiElement;
 import com.intellij.psi.PsiReference;
-import com.intellij.psi.util.PsiTreeUtil;
 import com.intellij.util.containers.ContainerUtil;
 import org.jetbrains.annotations.NotNull;
 
@@ -40,7 +38,7 @@ protected GoVisitor buildGoVisitor(@NotNull final ProblemsHolder holder,
       @Override
       public void visitShortVarDeclaration(@NotNull GoShortVarDeclaration o) {
         List<GoVarDefinition> list = o.getVarDefinitionList();
-        if (list.isEmpty() || PsiTreeUtil.getParentOfType(o, GoForClause.class) != null) return;
+        if (list.isEmpty()) return;
 
         GoVarDefinition first = ContainerUtil.getFirstItem(list);
         GoVarDefinition last = ContainerUtil.getLastItem(list);

testData/highlighting/shortVars.go

@@ -65,10 +65,4 @@ func Test2() (err error) {
     }
 
     return err
-}
-
-func <warning>shadowing</warning>(i int) {
-    for i := 1; i < 3; i++ {
-        _ = i
-    }
-}
+}