From c7847bfd571a6c241ee99e22f18513389aa2376c Mon Sep 17 00:00:00 2001 From: Roman Shevchenko Date: Fri, 23 Oct 2015 19:48:19 +0200 Subject: [PATCH] Cleanup (minor optimization) --- .../com/intellij/ide/plugins/InstalledPluginsState.java | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/platform/platform-impl/src/com/intellij/ide/plugins/InstalledPluginsState.java b/platform/platform-impl/src/com/intellij/ide/plugins/InstalledPluginsState.java index 785bfcfb33c5..d67bd8f891e1 100644 --- a/platform/platform-impl/src/com/intellij/ide/plugins/InstalledPluginsState.java +++ b/platform/platform-impl/src/com/intellij/ide/plugins/InstalledPluginsState.java @@ -19,15 +19,14 @@ import com.intellij.idea.IdeaApplication; import com.intellij.openapi.components.ServiceManager; import com.intellij.openapi.extensions.PluginId; import com.intellij.openapi.updateSettings.impl.PluginDownloader; -import com.intellij.util.SmartList; import com.intellij.util.containers.ContainerUtil; import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable; import java.util.Collection; import java.util.Collections; -import java.util.List; import java.util.Map; +import java.util.Set; /** * A service to hold a state of plugin changes in a current session (i.e. before the changes are applied on restart). @@ -45,7 +44,7 @@ public class InstalledPluginsState { private final Object myLock = new Object(); private final Map myInstalledPlugins = ContainerUtil.newIdentityHashMap(); private final Map myUpdatedPlugins = ContainerUtil.newIdentityHashMap(); - private final List myOutdatedPlugins = new SmartList(); + private final Set myOutdatedPlugins = ContainerUtil.newHashSet(); @NotNull public Collection getInstalledPlugins() { @@ -90,9 +89,7 @@ public class InstalledPluginsState { synchronized (myLock) { if (supersedes) { - if (!myOutdatedPlugins.contains(idString)) { - myOutdatedPlugins.add(idString); - } + myOutdatedPlugins.add(idString); } else { myOutdatedPlugins.remove(idString);