cleanup
authorAlexey Kudravtsev <cdr@intellij.com>
Tue, 5 Sep 2017 16:05:39 +0000 (19:05 +0300)
committerAlexey Kudravtsev <cdr@intellij.com>
Wed, 6 Sep 2017 10:29:58 +0000 (13:29 +0300)
platform/analysis-api/src/com/intellij/psi/search/scope/packageSet/NamedScopesHolder.java
platform/lang-api/src/com/intellij/facet/Facet.java
platform/lang-api/src/com/intellij/facet/ModifiableFacetModel.java
platform/lang-impl/src/com/intellij/facet/FacetManagerImpl.java
platform/lang-impl/src/com/intellij/facet/impl/FacetModelImpl.java
platform/projectModel-impl/src/com/intellij/openapi/roots/impl/libraries/LibraryTableBase.java
python/pluginJava/com/jetbrains/python/facet/PythonFacet.java
python/pluginMinor/com/jetbrains/python/minor/facet/PythonFacet.java

index fc31811426e3abcff500ef0f3ace00453a24e4e4..e7dfebf0c1e57e213f19ba8f6b0b86faaeb0b275 100644 (file)
@@ -47,8 +47,9 @@ public abstract class NamedScopesHolder implements PersistentStateComponent<Elem
   public abstract String getDisplayName();
 
   public abstract Icon getIcon();
-  public interface ScopeListener {
 
+  @FunctionalInterface
+  public interface ScopeListener {
     void scopesChanged();
   }
 
index f0c95764f27af42243ca70be8ab8067d61c79461..58182953c8c38754d34f40400e0b6ab3fc40e532 100644 (file)
@@ -102,7 +102,7 @@ public class Facet<C extends FacetConfiguration> extends UserDataHolderBase impl
     return super.hashCode();
   }
 
-  @SuppressWarnings({"EqualsWhichDoesntCheckParameterClass"})
+  @SuppressWarnings("EqualsWhichDoesntCheckParameterClass")
   public final boolean equals(Object obj) {
     return super.equals(obj);
   }
@@ -113,7 +113,7 @@ public class Facet<C extends FacetConfiguration> extends UserDataHolderBase impl
   }
 
   /**
-   * Use {@link com.intellij.facet.ModifiableFacetModel#rename} to rename facets
+   * Use {@link ModifiableFacetModel#rename} to rename facets
    */
   final void setName(@NotNull final String name) {
     myName = name;
index 550899d47ea0da14db82bda052cf738bf58c8677..4664e52adbadfd4f9cff89f22ee675c2217d1a6d 100644 (file)
@@ -41,8 +41,9 @@ public interface ModifiableFacetModel extends FacetModel {
 
   boolean isNewFacet(Facet facet);
 
-  void addListener(@NotNull Listener listener, @Nullable Disposable parentDisposable);
+  void addListener(@NotNull Listener listener, @NotNull Disposable parentDisposable);
 
+  @FunctionalInterface
   interface Listener {
     void onChanged();
   }
index d788090b9e6cd20004cb281ff5040220c36ca113..98d91c6da3ad55b8e80080cf0f195b07fe2b7a29 100644 (file)
@@ -62,7 +62,7 @@ public class FacetManagerImpl extends FacetManager implements ModuleComponent, P
 
   private final Module myModule;
   private final FacetManagerModel myModel = new FacetManagerModel();
-  private boolean myInsideCommit = false;
+  private boolean myInsideCommit;
   private final MessageBus myMessageBus;
   private boolean myModuleAdded;
 
@@ -513,7 +513,7 @@ public class FacetManagerImpl extends FacetManager implements ModuleComponent, P
 
   private static class FacetManagerModel extends FacetModelBase {
     private Facet[] myAllFacets = Facet.EMPTY_ARRAY;
-    private Set<ProjectModelExternalSource> myExternalSources = new LinkedHashSet<>();
+    private final Set<ProjectModelExternalSource> myExternalSources = new LinkedHashSet<>();
 
     @Override
     @NotNull
@@ -521,7 +521,7 @@ public class FacetManagerImpl extends FacetManager implements ModuleComponent, P
       return myAllFacets;
     }
 
-    public void setAllFacets(final Facet[] allFacets) {
+    void setAllFacets(final Facet[] allFacets) {
       myExternalSources.clear();
       for (Facet facet : allFacets) {
         ContainerUtil.addIfNotNull(myExternalSources, facet.getExternalSource());
@@ -536,7 +536,7 @@ public class FacetManagerImpl extends FacetManager implements ModuleComponent, P
     private final String myOldName;
     private final String myNewName;
 
-    public FacetRenameInfo(final Facet facet, final String oldName, final String newName) {
+    FacetRenameInfo(final Facet facet, final String oldName, final String newName) {
       myFacet = facet;
       myOldName = oldName;
       myNewName = newName;
index 0cd0ae762cdee1ec01344360a7a7572c38b0873d..e323f442c73d9a87d6a7a26b056675eb312cc1a5 100644 (file)
@@ -119,16 +119,9 @@ public class FacetModelImpl extends FacetModelBase implements ModifiableFacetMod
   }
 
   @Override
-  public void addListener(@NotNull final Listener listener, @Nullable Disposable parentDisposable) {
+  public void addListener(@NotNull final Listener listener, @NotNull Disposable parentDisposable) {
     myListeners.add(listener);
-    if (parentDisposable != null) {
-      Disposer.register(parentDisposable, new Disposable() {
-        @Override
-        public void dispose() {
-          myListeners.remove(listener);
-        }
-      });
-    }
+    Disposer.register(parentDisposable, () -> myListeners.remove(listener));
   }
 
   @Override
index bc0165034e8d0c03ba47edfc90da22744682d6be..dbff198a33801602d077203cb929a9c5e1405d0d 100644 (file)
@@ -146,6 +146,7 @@ public abstract class LibraryTableBase implements PersistentStateComponent<Eleme
     }
   }
 
+  @NotNull
   @Override
   public Library createLibrary() {
     ApplicationManager.getApplication().assertWriteAccessAllowed();
@@ -164,6 +165,7 @@ public abstract class LibraryTableBase implements PersistentStateComponent<Eleme
     myModificationCount++;
   }
 
+  @NotNull
   @Override
   public Library createLibrary(String name) {
     final ModifiableModel modifiableModel = getModifiableModel();
@@ -292,16 +294,19 @@ public abstract class LibraryTableBase implements PersistentStateComponent<Eleme
       LOG.assertTrue(myWritable);
     }
 
+    @NotNull
     @Override
     public Library createLibrary(String name) {
       return createLibrary(name, null);
     }
 
+    @NotNull
     @Override
     public Library createLibrary(String name, @Nullable PersistentLibraryKind kind) {
       return createLibrary(name, kind, null);
     }
 
+    @NotNull
     @Override
     public Library createLibrary(String name, @Nullable PersistentLibraryKind kind, @Nullable ProjectModelExternalSource externalSource) {
       assertWritable();
index defc69eb966b1fc9783a2839e3e35753842b7466..8ed4cd9d5446a5fe225ad94219fe1fad6a116f3f 100644 (file)
@@ -29,16 +29,18 @@ public class PythonFacet extends LibraryContributingFacet<PythonFacetConfigurati
 
   public PythonFacet(@NotNull final FacetType facetType,
                      @NotNull final Module module,
-                     final @NotNull String name,
+                     @NotNull final String name,
                      @NotNull final PythonFacetConfiguration configuration,
                      Facet underlyingFacet) {
     super(facetType, module, name, configuration, underlyingFacet);
   }
 
+  @Override
   public void updateLibrary() {
     PythonFacetUtil.updateLibrary(getModule(), getConfiguration());
   }
 
+  @Override
   public void removeLibrary() {
     PythonFacetUtil.removeLibrary(getModule());
   }
@@ -47,6 +49,7 @@ public class PythonFacet extends LibraryContributingFacet<PythonFacetConfigurati
     return sdkName + PYTHON_FACET_LIBRARY_NAME_SUFFIX;
   }
 
+  @Override
   public void initFacet() {
     updateLibrary();
   }
index ab37a5da81012bd8523ef832ef38817873ed21ff..a44865c1353929114f6dce3ea4ec9c64304f11d0 100644 (file)
@@ -31,19 +31,22 @@ import org.jetbrains.annotations.NotNull;
 public class PythonFacet extends LibraryContributingFacet<PythonFacetType.PythonFacetConfiguration> {
   public static final FacetTypeId<PythonFacet> ID = new FacetTypeId<>("python");
 
-  public PythonFacet(@NotNull final FacetType facetType, @NotNull final Module module, final @NotNull String name, @NotNull final PythonFacetType.PythonFacetConfiguration configuration,
+  public PythonFacet(@NotNull final FacetType facetType, @NotNull final Module module, @NotNull final String name, @NotNull final PythonFacetType.PythonFacetConfiguration configuration,
                      Facet underlyingFacet) {
     super(facetType, module, name, configuration, underlyingFacet);
   }
 
+  @Override
   public void updateLibrary() {
     PythonFacetUtil.updateLibrary(getModule(), getConfiguration());
   }
 
+  @Override
   public void removeLibrary() {
     PythonFacetUtil.removeLibrary(getModule());
   }
 
+  @Override
   public void initFacet() {
     updateLibrary();
   }