IG: add assertions instead of silently ignoring problems
authorBas Leijdekkers <basleijdekkers@gmail.com>
Tue, 13 Sep 2016 10:46:09 +0000 (12:46 +0200)
committerBas Leijdekkers <basleijdekkers@gmail.com>
Tue, 13 Sep 2016 11:03:09 +0000 (13:03 +0200)
plugins/InspectionGadgets/InspectionGadgetsAnalysis/src/com/siyeh/ig/BaseInspection.java
plugins/InspectionGadgets/InspectionGadgetsAnalysis/src/com/siyeh/ig/BaseInspectionVisitor.java

index 67b1f9e1ec43ce22f19a65340a9c470679aab595..404f909835975f82f58fbdd966c15d196ea1d223 100644 (file)
@@ -118,9 +118,10 @@ public abstract class BaseInspection extends BaseJavaBatchLocalInspectionTool {
 
   @Override
   @NotNull
-  public final PsiElementVisitor buildVisitor(@NotNull ProblemsHolder holder,
-                                        boolean isOnTheFly) {
-    if (!shouldInspect(holder.getFile())) {
+  public final PsiElementVisitor buildVisitor(@NotNull ProblemsHolder holder, boolean isOnTheFly) {
+    final PsiFile file = holder.getFile();
+    assert file.isPhysical();
+    if (!shouldInspect(file)) {
       return new PsiElementVisitor() { };
     }
     final BaseInspectionVisitor visitor = buildVisitor();
index 4d4d8aae6fbff6c51f2600cef43835720aa55611..4bcd01fd35367ed5de5ced94d0adcfd242aecd61 100644 (file)
@@ -96,10 +96,7 @@ public abstract class BaseInspectionVisitor extends JavaElementVisitor {
     else {
       nameIdentifier = aClass.getNameIdentifier();
     }
-    if (nameIdentifier != null && !nameIdentifier.isPhysical()) {
-      nameIdentifier = nameIdentifier.getNavigationElement();
-    }
-    if (nameIdentifier == null || !nameIdentifier.isPhysical()) {
+    if (nameIdentifier == null) {
       registerError(aClass.getContainingFile(), infos);
     }
     else {
@@ -182,9 +179,7 @@ public abstract class BaseInspectionVisitor extends JavaElementVisitor {
   protected final void registerError(@NotNull PsiElement location,
                                      final ProblemHighlightType highlightType,
                                      Object... infos) {
-    if (!location.isPhysical() || location.getTextLength() == 0 && !(location instanceof PsiFile)) {
-      return;
-    }
+    assert location.getTextLength() != 0 || location instanceof PsiFile;
     final LocalQuickFix[] fixes = createAndInitFixes(infos);
     final String description = inspection.buildErrorString(infos);
     holder.registerProblem(location, description, highlightType, fixes);
@@ -197,9 +192,7 @@ public abstract class BaseInspectionVisitor extends JavaElementVisitor {
   protected final void registerErrorAtOffset(@NotNull PsiElement location, int offset, int length,
                                              ProblemHighlightType highlightType,
                                              Object... infos) {
-    if (location.getTextLength() == 0 || length == 0) {
-      return;
-    }
+    assert !(location.getTextLength() == 0 || length == 0);
     final LocalQuickFix[] fixes = createAndInitFixes(infos);
     final String description = inspection.buildErrorString(infos);
     final TextRange range = new TextRange(offset, offset + length);