Revert: annotations: check error elements after annotation to rise annotation not expected here instead of some unpredictable warnings (IDEA-160106) (d4e7271498)

java/java-psi-api/src/com/intellij/codeInsight/AnnotationTargetUtil.java

@@ -61,9 +61,6 @@ public class AnnotationTargetUtil {
     }
 
     if (owner instanceof PsiModifierList) {
-      if (((PsiModifierList)owner).getNextSibling() instanceof PsiErrorElement) {
-        return TargetType.EMPTY_ARRAY;
-      }
       PsiElement element = ((PsiModifierList)owner).getParent();
       if (element instanceof PsiPackageStatement) {
         return PACKAGE_TARGETS;

java/java-psi-impl/src/com/intellij/psi/impl/PsiClassImplUtil.java

@@ -467,8 +467,8 @@ public class PsiClassImplUtil {
                                                     @NotNull LanguageLevel languageLevel,
                                                     boolean isRaw,
                                                     @NotNull GlobalSearchScope resolveScope) {
-    if (last instanceof PsiTypeParameterList || last instanceof PsiModifierList && !(last.getNextSibling() instanceof PsiErrorElement)) {
-      return true;
+    if (last instanceof PsiTypeParameterList || last instanceof PsiModifierList) {
+      return true; //TypeParameterList and ModifierList do not see our declarations
     }
     if (visited != null && visited.contains(aClass)) return true;
 

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/annotations/annotationOverIncompleteCode.java

@@ -1,12 +0,0 @@
-class Dummy {
-
-  public @interface Debug {
-    String value() default "[no comment]";
-  }
-
-  private static final class Constants {
-    private static final String INPUT = "Input";
-  }
-
-  <error descr="Annotations are not allowed here">@Dummy.Debug(Constants.INPUT)</error><EOLError descr="Identifier or type expected"></EOLError>
-}

java/java-tests/testSrc/com/intellij/codeInsight/daemon/AnnotationsHighlightingTest.java

@@ -49,7 +49,6 @@ public class AnnotationsHighlightingTest extends LightDaemonAnalyzerTestCase {
   public void testRepeatable() { doTest8(); }
   public void testEnumValues() { doTest8(); }
   public void testReceiverParameters() { doTest8(); }
-  public void testAnnotationOverIncompleteCode() { doTest(); }
 
   private void doTest() {
     setLanguageLevel(LanguageLevel.JDK_1_7);