cleanup
authorAlexey Kudravtsev <cdr@intellij.com>
Wed, 28 Nov 2018 16:59:46 +0000 (19:59 +0300)
committerAlexey Kudravtsev <cdr@intellij.com>
Thu, 29 Nov 2018 10:24:54 +0000 (13:24 +0300)
15 files changed:
java/compiler/impl/src/com/intellij/compiler/impl/CompositeScope.java
java/compiler/impl/src/com/intellij/compiler/impl/FileSetCompileScope.java
java/compiler/impl/src/com/intellij/compiler/impl/ModuleCompileScope.java
java/compiler/impl/src/com/intellij/compiler/impl/OneProjectItemCompileScope.java
java/compiler/impl/src/com/intellij/compiler/impl/ProjectCompileScope.java
java/compiler/impl/src/com/intellij/compiler/options/JavaCompilersTab.java
java/compiler/openapi/src/com/intellij/openapi/compiler/CompileScope.java
java/java-impl/src/com/intellij/cyclicDependencies/CyclicDependenciesBuilder.java
java/java-impl/src/com/intellij/ide/projectView/impl/nodes/PackageElement.java
platform/analysis-api/src/com/intellij/psi/search/GlobalSearchScopesCore.java
platform/core-api/src/com/intellij/psi/search/GlobalSearchScope.java
platform/core-api/src/com/intellij/psi/search/LocalSearchScope.java
platform/core-impl/src/com/intellij/psi/impl/source/PsiFileImpl.java
platform/lang-impl/src/com/intellij/packageDependencies/actions/AnalyzeDependenciesHandler.java
platform/lang-impl/src/com/intellij/packageDependencies/actions/BackwardDependenciesHandler.java

index b9882d3df577fc650537266e58021d5f78667c7b..f168726d9c38ad89f53de5e75518916ddfe1655f 100644 (file)
@@ -72,7 +72,7 @@ public class CompositeScope extends ExportableUserDataHolderBase implements Comp
   }
 
   @Override
-  public boolean belongs(String url) {
+  public boolean belongs(@NotNull String url) {
     for (CompileScope scope : myScopes) {
       if (scope.belongs(url)) {
         return true;
index 1e06f08ec223bf53d40b3e06efedded625c4fb46..f91e2b93757117e42b1211e043d2a2730f2375d4 100644 (file)
@@ -34,7 +34,7 @@ import java.util.*;
 public class FileSetCompileScope extends ExportableUserDataHolderBase implements CompileScope {
   private final Set<VirtualFile> myRootFiles = new HashSet<>();
   private final Set<String> myDirectoryUrls = new HashSet<>();
-  private Set<String> myUrls = null; // urls caching
+  private Set<String> myUrls; // urls caching
   private final Module[] myAffectedModules;
 
   public FileSetCompileScope(final Collection<VirtualFile> files, Module[] modules) {
@@ -82,7 +82,7 @@ public class FileSetCompileScope extends ExportableUserDataHolderBase implements
   }
 
   @Override
-  public boolean belongs(String url) {
+  public boolean belongs(@NotNull String url) {
     //url = CompilerUtil.normalizePath(url, '/');
     if (getUrls().contains(url)) {
       return true;
index 73535145826af7948b79aefd6b543adee6e40e61..47a26739a95c739fe8fa50e8f1c2a6feca41d2d2 100644 (file)
@@ -97,7 +97,7 @@ public class ModuleCompileScope extends FileIndexCompileScope {
   }
 
   @Override
-  public boolean belongs(final String url) {
+  public boolean belongs(@NotNull final String url) {
     if (myScopeModules.isEmpty() && myIncludedUnloadedModules.isEmpty()) {
       return false; // optimization
     }
index 1767a5699f5754a1f33d3af799a3f66db0293bc3..84a65ce04e113177958b143667f3050921ccd216 100644 (file)
@@ -62,7 +62,7 @@ public class OneProjectItemCompileScope extends ExportableUserDataHolderBase imp
   }
 
   @Override
-  public boolean belongs(String url) {
+  public boolean belongs(@NotNull String url) {
     if (myFile.isDirectory()){
       return FileUtil.startsWith(url, myUrl);
     }
index f6f060d2b9eebe5387c3491cbddf4c4f89a0e666..cbf4ab1451230c356facde98a2f5ffed9a17bc1e 100644 (file)
@@ -42,7 +42,7 @@ public class ProjectCompileScope extends FileIndexCompileScope {
   }
 
   @Override
-  public boolean belongs(String url) {
+  public boolean belongs(@NotNull String url) {
     final VirtualFile file = VirtualFileManager.getInstance().findFileByUrl(url);
     if (file != null) {
       for (FileIndex index : getFileIndices()) {
index b78268f35ba19b93a8c393ddb9cb0761aa569a6b..4f532a05e7a64565c27fe675b465890673e91198 100644 (file)
@@ -1,7 +1,6 @@
 // Copyright 2000-2017 JetBrains s.r.o. Use of this source code is governed by the Apache 2.0 license that can be found in the LICENSE file.
 package com.intellij.compiler.options;
 
-import com.intellij.codeInsight.daemon.DaemonCodeAnalyzer;
 import com.intellij.compiler.CompilerConfiguration;
 import com.intellij.compiler.CompilerConfigurationImpl;
 import com.intellij.compiler.impl.javaCompiler.BackendCompiler;
@@ -149,7 +148,6 @@ public class JavaCompilersTab implements SearchableConfigurable, Configurable.No
     finally {
       BuildManager.getInstance().clearState(myProject);
       PsiManager.getInstance(myProject).dropPsiCaches();
-      DaemonCodeAnalyzer.getInstance(myProject).restart();
     }
   }
 
index 1517120bce6c480c0634bcbb5406ed21e7392d01..9c8766e5c5cd1a5b656054fdac0c855baf891431 100644 (file)
@@ -50,7 +50,7 @@ public interface CompileScope extends ExportableUserDataHolder {
    * @return true if the url specified belongs to the scope, false otherwise.
    *         Note: the method may be time-consuming.
    */
-  boolean belongs(String url);
+  boolean belongs(@NotNull String url);
 
   /**
    * Returns the list of modules files in which belong to the scope.
index dec004093f0567cd697687b16691adab0e4963ab..aec2bfc91b29b0ae9ed7e2bc8a604426c84c933f 100644 (file)
@@ -32,6 +32,7 @@ import org.jetbrains.annotations.NotNull;
 import java.util.*;
 
 public class CyclicDependenciesBuilder{
+  @NotNull
   private final Project myProject;
   private final AnalysisScope myScope;
   private final Map<String, PsiPackage> myPackages = new HashMap<>();
@@ -45,7 +46,7 @@ public class CyclicDependenciesBuilder{
 
   private String myRootNodeNameInUsageView;
 
-  public CyclicDependenciesBuilder(final Project project, final AnalysisScope scope) {
+  public CyclicDependenciesBuilder(@NotNull Project project, final AnalysisScope scope) {
     myProject = project;
     myScope = scope;
     myForwardBuilder = new ForwardDependenciesBuilder(myProject, myScope){
@@ -69,6 +70,7 @@ public class CyclicDependenciesBuilder{
     myRootNodeNameInUsageView = rootNodeNameInUsageView;
   }
 
+  @NotNull
   public Project getProject() {
     return myProject;
   }
index 837f6f7c922d33aeab06261b79c8e07e4a4c5bbd..c3a1ee1f75f68b46c6751322212f346b5a6b2f03 100644 (file)
@@ -67,6 +67,7 @@ public final class PackageElement implements Queryable, RootsProvider {
     return roots;
   }
 
+  @Override
   public boolean equals(Object o) {
     if (this == o) return true;
     if (!(o instanceof PackageElement)) return false;
@@ -80,10 +81,10 @@ public final class PackageElement implements Queryable, RootsProvider {
     return true;
   }
 
+  @Override
   public int hashCode() {
-    int result;
-    result = (myModule != null ? myModule.hashCode() : 0);
-    result = 29 * result + (myElement.hashCode());
+    int result = myModule != null ? myModule.hashCode() : 0;
+    result = 29 * result + myElement.hashCode();
     result = 29 * result + (myIsLibraryElement ? 1 : 0);
     return result;
   }
index 18274ca81b0fbf939d0e38c386b56cab4e753518..1fb5b95fe0202d08fdf4e0e318c4ceeee5c4d4de 100644 (file)
@@ -287,6 +287,7 @@ public class GlobalSearchScopesCore {
       return super.uniteWith(scope);
     }
 
+    @NotNull
     private static Set<VirtualFile> union(boolean addDir1, @NotNull VirtualFile dir1, boolean addDir2, @NotNull VirtualFile dir2) {
       if (addDir1 && addDir2) return ContainerUtil.newHashSet(dir1, dir2);
       if (addDir1) return Collections.singleton(dir1);
index 4ae7d6f54b430597f9f2a1ec74156ae828102021..af1383f19842af7a93d44d8ae5399482ef378b83 100644 (file)
@@ -839,7 +839,7 @@ public abstract class GlobalSearchScope extends SearchScope implements ProjectAw
         Project project = getProject();
         myHasFilesOutOfProjectRoots = result =
           project != null && !project.isDefault() &&
-          myFiles.stream().anyMatch(file -> FileIndexFacade.getInstance(project).getModuleForFile(file) == null);
+          ContainerUtil.find(myFiles, file -> FileIndexFacade.getInstance(project).getModuleForFile(file) != null) == null;
       }
       return result;
     }
index ef88b50be32421337660fda8bac6c6981be02b4d..14ab5985fc19a5c9a8fa586d4dcce1334f60d38d 100644 (file)
@@ -111,6 +111,7 @@ public class LocalSearchScope extends SearchScope {
     return myVirtualFiles;
   }
 
+  @Override
   public boolean equals(Object o) {
     if (this == o) return true;
     if (!(o instanceof LocalSearchScope)) return false;
@@ -195,6 +196,7 @@ public class LocalSearchScope extends SearchScope {
     return null;
   }
 
+  @Override
   public String toString() {
     StringBuilder result = new StringBuilder();
     for (int i = 0; i < myScope.length; i++) {
@@ -257,11 +259,11 @@ public class LocalSearchScope extends SearchScope {
     return commonParent;
   }
 
-  public boolean isInScope(VirtualFile file) {
+  public boolean isInScope(@NotNull VirtualFile file) {
     return ArrayUtil.indexOf(myVirtualFiles, file) != -1;
   }
 
-  public boolean containsRange(PsiFile file, @NotNull TextRange range) {
+  public boolean containsRange(@NotNull PsiFile file, @NotNull TextRange range) {
     for (PsiElement element : getScope()) {
       if (file == element.getContainingFile() && element.getTextRange().contains(range)) {
         return true;
@@ -291,8 +293,5 @@ public class LocalSearchScope extends SearchScope {
              ? EMPTY
              : new LocalSearchScope(PsiUtilCore.toPsiElementArray(result), scope.getDisplayName(), scope.isIgnoreInjectedPsi());
     });
-
-
-
   }
 }
index 80f14807e10847ad472f51812e7798487bc8f57f..22fcf376eaeb617171930b04349a9503091c0da1 100644 (file)
@@ -115,7 +115,7 @@ public abstract class PsiFileImpl extends ElementBase implements PsiFileEx, PsiF
     return null;
   }
 
-  protected FileElement derefTreeElement() {
+  FileElement derefTreeElement() {
     return myTrees.derefTreeElement();
   }
 
@@ -337,7 +337,7 @@ public abstract class PsiFileImpl extends ElementBase implements PsiFileEx, PsiF
   }
 
   @Override
-  @SuppressWarnings({"CloneDoesntCallSuperClone"})
+  @SuppressWarnings("CloneDoesntCallSuperClone")
   protected PsiFileImpl clone() {
     FileViewProvider viewProvider = getViewProvider();
     FileViewProvider providerCopy = viewProvider.clone();
@@ -990,7 +990,7 @@ public abstract class PsiFileImpl extends ElementBase implements PsiFileEx, PsiF
     putInfo(this, info);
   }
 
-  public static void putInfo(PsiFile psiFile, Map<String, String> info) {
+  public static void putInfo(@NotNull PsiFile psiFile, @NotNull Map<String, String> info) {
     info.put("fileName", psiFile.getName());
     info.put("fileType", psiFile.getFileType().toString());
   }
index efa5a9fa4b01d04a440053d106480339548aa50d..a817fa823a565f400389c1a71ab9574c5c0f4960 100644 (file)
@@ -22,6 +22,7 @@ import com.intellij.openapi.project.Project;
 import com.intellij.packageDependencies.DependenciesBuilder;
 import com.intellij.packageDependencies.ForwardDependenciesBuilder;
 import com.intellij.psi.PsiFile;
+import org.jetbrains.annotations.NotNull;
 
 import java.util.Collections;
 import java.util.HashSet;
@@ -31,7 +32,7 @@ import java.util.Set;
 public class AnalyzeDependenciesHandler extends DependenciesHandlerBase {
   private final int myTransitiveBorder;
 
-  public AnalyzeDependenciesHandler(Project project, List<? extends AnalysisScope> scopes, int transitiveBorder, Set<PsiFile> excluded) {
+  public AnalyzeDependenciesHandler(@NotNull Project project, List<? extends AnalysisScope> scopes, int transitiveBorder, Set<PsiFile> excluded) {
     super(project, scopes, excluded);
     myTransitiveBorder = transitiveBorder;
   }
index c04aeed4d85493b47e291d7607bbd27ceb6ea99e..9ea5463604f8f786c07b19bd207f6d2eb6a0db5a 100644 (file)
@@ -22,6 +22,7 @@ import com.intellij.openapi.project.Project;
 import com.intellij.packageDependencies.BackwardDependenciesBuilder;
 import com.intellij.packageDependencies.DependenciesBuilder;
 import com.intellij.psi.PsiFile;
+import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
 import java.util.Collections;
@@ -32,11 +33,11 @@ import java.util.Set;
 public class BackwardDependenciesHandler extends DependenciesHandlerBase {
   private final AnalysisScope myScopeOfInterest;
 
-  public BackwardDependenciesHandler(Project project, AnalysisScope scope, final AnalysisScope selectedScope) {
+  public BackwardDependenciesHandler(@NotNull Project project, AnalysisScope scope, final AnalysisScope selectedScope) {
     this(project, Collections.singletonList(scope), selectedScope, new HashSet<>());
   }
 
-  public BackwardDependenciesHandler(final Project project, final List<? extends AnalysisScope> scopes, final @Nullable AnalysisScope scopeOfInterest, Set<PsiFile> excluded) {
+  public BackwardDependenciesHandler(@NotNull Project project, final List<? extends AnalysisScope> scopes, @Nullable final AnalysisScope scopeOfInterest, Set<PsiFile> excluded) {
     super(project, scopes, excluded);
     myScopeOfInterest = scopeOfInterest;
   }