cleanup
authorVladimir Krivosheev <vladimir.krivosheev@jetbrains.com>
Wed, 13 Jul 2016 13:34:51 +0000 (15:34 +0200)
committerVladimir Krivosheev <vladimir.krivosheev@jetbrains.com>
Wed, 13 Jul 2016 13:34:51 +0000 (15:34 +0200)
java/java-analysis-impl/src/com/intellij/codeInspection/RedundantSuppressInspectionBase.java
java/java-tests/testSrc/com/intellij/codeInspection/SingleInspectionProfilePanelTest.java
java/java-tests/testSrc/com/intellij/codeInspection/ex/InspectionProfileTest.java
platform/analysis-api/src/com/intellij/codeInspection/ProblemDescriptionsProcessor.java
platform/analysis-impl/src/com/intellij/codeInspection/ex/InspectionProfileImpl.java

index df1fe781baa1d497b16ff0220848776174a0fd1e..3c2ff0ef3ea3c282499ba333a08bdf8a349f15bd 100644 (file)
@@ -224,29 +224,7 @@ public class RedundantSuppressInspectionBase extends GlobalInspectionTool {
           descriptors = new ArrayList<CommonProblemDescriptor>();
           globalContext.getRefManager().iterate(new RefVisitor() {
             @Override public void visitElement(@NotNull RefEntity refEntity) {
-              CommonProblemDescriptor[]
-                descriptors1 = global.getTool().checkElement(refEntity, scope, manager, globalContext, new ProblemDescriptionsProcessor() {
-                @Nullable
-                @Override
-                public CommonProblemDescriptor[] getDescriptions(@NotNull RefEntity refEntity) {
-                  return CommonProblemDescriptor.EMPTY_ARRAY;
-                }
-
-                @Override
-                public void ignoreElement(@NotNull RefEntity refEntity) {
-
-                }
-
-                @Override
-                public void addProblemElement(@Nullable RefEntity refEntity, @NotNull CommonProblemDescriptor... commonProblemDescriptors) {
-                  int i =0;
-                }
-
-                @Override
-                public RefEntity getElement(@NotNull CommonProblemDescriptor descriptor) {
-                  return null;
-                }
-              });
+              CommonProblemDescriptor[] descriptors1 = global.getTool().checkElement(refEntity, scope, manager, globalContext, new ProblemDescriptionsProcessor() {});
               if (descriptors1 != null) {
                 ContainerUtil.addAll(descriptors, descriptors1);
               }
index 74a986cafb0728bb8c276998a1392323ae0c8ba7..b155c551a1e4180ba91d31ef6ceb91cc1bd0daf7 100644 (file)
@@ -32,6 +32,8 @@ import org.jetbrains.annotations.NotNull;
  *         Date: 5/10/12
  */
 public class SingleInspectionProfilePanelTest extends LightIdeaTestCase {
+  private final JavaDocLocalInspection myInspection = new JavaDocLocalInspection();
+
   // see IDEA-85700
   public void testSettingsModification() throws Exception {
     Project project = ProjectManager.getInstance().getDefaultProject();
@@ -39,7 +41,7 @@ public class SingleInspectionProfilePanelTest extends LightIdeaTestCase {
     InspectionProfileImpl profile = (InspectionProfileImpl)profileManager.getProfile(PROFILE);
     profile.initInspectionTools(project);
 
-    InspectionProfileImpl model = (InspectionProfileImpl)profile.getModifiableModel();
+    InspectionProfileImpl model = profile.getModifiableModel();
     SingleInspectionProfilePanel panel = new SingleInspectionProfilePanel(profileManager, PROFILE, model, profile);
     panel.setVisible(true);
     panel.reset();
@@ -64,7 +66,7 @@ public class SingleInspectionProfilePanelTest extends LightIdeaTestCase {
     JavaDocLocalInspection originalTool = getInspection(profile);
     originalTool.myAdditionalJavadocTags = "foo";
 
-    InspectionProfileImpl model = (InspectionProfileImpl)profile.getModifiableModel();
+    InspectionProfileImpl model = profile.getModifiableModel();
 
     SingleInspectionProfilePanel panel = new SingleInspectionProfilePanel(profileManager, PROFILE, model, profile);
     panel.setVisible(true);
@@ -91,7 +93,7 @@ public class SingleInspectionProfilePanelTest extends LightIdeaTestCase {
     JavaDocLocalInspection originalTool = getInspection(profile);
     assertEquals("", originalTool.myAdditionalJavadocTags);
 
-    InspectionProfileImpl model = (InspectionProfileImpl)profile.getModifiableModel();
+    InspectionProfileImpl model = profile.getModifiableModel();
     JavaDocLocalInspection copyTool = getInspection(model);
     copyTool.myAdditionalJavadocTags = "foo";
     // this change IS NOT COMMITTED
@@ -117,8 +119,6 @@ public class SingleInspectionProfilePanelTest extends LightIdeaTestCase {
     super.tearDown();
   }
 
-  private final JavaDocLocalInspection myInspection = new JavaDocLocalInspection();
-
   @NotNull
   @Override
   protected LocalInspectionTool[] configureLocalInspectionTools() {
index da839ee62edbe127f51f046c95836f1dc11b4991..821daec5d5842185a72a239d6a21ed41b3f6d8ff 100644 (file)
@@ -319,7 +319,7 @@ public class InspectionProfileTest extends LightIdeaTestCase {
                                                "    <option name=\"ADD_NONJAVA_TO_ENTRIES\" value=\"false\" />\n" +
                                                "  </inspection_tool>\n" +
                                                "</profile>").getRootElement());
-    InspectionProfileImpl model = (InspectionProfileImpl)profile.getModifiableModel();
+    InspectionProfileImpl model = profile.getModifiableModel();
     InspectionToolWrapper toolWrapper = model.getInspectionTool("unused", getProject());
     UnusedDeclarationInspectionBase tool = (UnusedDeclarationInspectionBase)toolWrapper.getTool();
     UnusedSymbolLocalInspectionBase inspectionTool = tool.getSharedLocalInspectionTool();
@@ -439,7 +439,7 @@ public class InspectionProfileTest extends LightIdeaTestCase {
     assertTrue(profile.isToolEnabled(HighlightDisplayKey.find("foo")));
     assertTrue(profile.getToolDefaultState("foo", getProject()).isEnabled());
 
-    InspectionProfileImpl model = (InspectionProfileImpl)profile.getModifiableModel();
+    InspectionProfileImpl model = profile.getModifiableModel();
     model.lockProfile(true);
     model.initInspectionTools(getProject()); // todo commit should take care of initialization
     model.commit();
index aefd6b0190f1ce35376c1ba37c4f5cdb5c88bc03..ceb0ca61b9961cb5a00258510208fc5e7bc66085 100644 (file)
@@ -35,7 +35,7 @@ public interface ProblemDescriptionsProcessor {
    */
   @Nullable
   default CommonProblemDescriptor[] getDescriptions(@NotNull RefEntity refEntity) {
-    return new CommonProblemDescriptor[0];
+    return CommonProblemDescriptor.EMPTY_ARRAY;
   }
 
   /**
index a1cbc085f55c75eddcd07e3ecf500dcc317d9680..7c28378ea31afa286228e014af60e0338d546405 100644 (file)
@@ -685,7 +685,7 @@ public class InspectionProfileImpl extends ProfileEx implements ModifiableModel,
 
   @Override
   @NotNull
-  public ModifiableModel getModifiableModel() {
+  public InspectionProfileImpl getModifiableModel() {
     return new InspectionProfileImpl(this);
   }