Cleanup (signatures)
authorRoman Shevchenko <roman.shevchenko@jetbrains.com>
Thu, 11 Dec 2014 17:19:18 +0000 (18:19 +0100)
committerRoman Shevchenko <roman.shevchenko@jetbrains.com>
Mon, 15 Dec 2014 18:55:04 +0000 (19:55 +0100)
platform/platform-impl/src/com/intellij/ide/plugins/InstalledPluginsTableModel.java
platform/platform-impl/src/com/intellij/ide/plugins/PluginManagerMain.java
platform/platform-impl/src/com/intellij/ide/plugins/PluginTableModel.java
platform/platform-impl/src/com/intellij/ide/plugins/UninstallPluginAction.java

index 9021029be77ff99984004340ca90e4b2a89dc493..7181fdfc802f41541af04e0df24765d55f5d0ff3 100644 (file)
@@ -267,8 +267,8 @@ public class InstalledPluginsTableModel extends PluginTableModel {
   }
 
   @Override
-  protected ArrayList<IdeaPluginDescriptor> toProcess() {
-    ArrayList<IdeaPluginDescriptor> toProcess = super.toProcess();
+  protected List<IdeaPluginDescriptor> toProcess() {
+    List<IdeaPluginDescriptor> toProcess = super.toProcess();
     for (IdeaPluginDescriptor descriptor : myInstalled) {
       if (!toProcess.contains(descriptor)) {
         toProcess.add(descriptor);
index f4d86210960689b8bc6b567c9f0844e84925f021..c4daefcae429242d5064f3c7f2728f2630beaf32 100644 (file)
@@ -273,7 +273,7 @@ public abstract class PluginManagerMain implements Disposable {
     requireShutdown |= val;
   }
 
-  public ArrayList<IdeaPluginDescriptorImpl> getDependentList(IdeaPluginDescriptorImpl pluginDescriptor) {
+  public List<IdeaPluginDescriptorImpl> getDependentList(IdeaPluginDescriptorImpl pluginDescriptor) {
     return pluginsModel.dependent(pluginDescriptor);
   }
 
index 4aac4e31573a6079a6c327aecbb81e3be7a3b253..4edabb8122e15541692a2d30541daaa4e3975660 100644 (file)
@@ -113,8 +113,8 @@ abstract public class PluginTableModel extends AbstractTableModel implements Sor
     fireTableCellUpdated(rowIndex, columnIndex);
   }
 
-  public ArrayList<IdeaPluginDescriptorImpl> dependent(IdeaPluginDescriptorImpl plugin) {
-    ArrayList<IdeaPluginDescriptorImpl> list = new ArrayList<IdeaPluginDescriptorImpl>();
+  public List<IdeaPluginDescriptorImpl> dependent(IdeaPluginDescriptorImpl plugin) {
+    List<IdeaPluginDescriptorImpl> list = new ArrayList<IdeaPluginDescriptorImpl>();
     for (IdeaPluginDescriptor any : getAllPlugins()) {
       if (any instanceof IdeaPluginDescriptorImpl) {
         PluginId[] dep = any.getDependentPluginIds();
@@ -160,7 +160,7 @@ abstract public class PluginTableModel extends AbstractTableModel implements Sor
     filter(desc);
   }
 
-  protected ArrayList<IdeaPluginDescriptor> toProcess() {
+  protected List<IdeaPluginDescriptor> toProcess() {
     return new ArrayList<IdeaPluginDescriptor>(view);
   }
 
index 1bd631f11d1cdff4ca664fc2e809063e99c957ba..369c6a2f65a32c954c19dabfd58802140375e588 100644 (file)
@@ -23,11 +23,9 @@ import com.intellij.openapi.actionSystem.Presentation;
 import com.intellij.openapi.extensions.PluginId;
 import com.intellij.openapi.project.DumbAware;
 import com.intellij.openapi.ui.Messages;
-import com.intellij.openapi.util.JDOMExternalizableStringList;
 
 import java.io.IOException;
-import java.util.ArrayList;
-import java.util.Set;
+import java.util.List;
 
 /**
  * @author Konstantin Bulenkov
@@ -93,7 +91,7 @@ public class UninstallPluginAction extends AnAction implements DumbAware {
 
       //  Get the list of plugins which depend on this one. If this list is
       //  not empty - issue warning instead of simple prompt.
-      ArrayList<IdeaPluginDescriptorImpl> dependant = host.getDependentList(pluginDescriptor);
+      List<IdeaPluginDescriptorImpl> dependant = host.getDependentList(pluginDescriptor);
       if (dependant.size() > 0) {
         message = IdeBundle.message("several.plugins.depend.on.0.continue.to.remove", pluginDescriptor.getName());
         actualDelete = (Messages.showYesNoDialog(host.getMainPanel(), message, IdeBundle.message("title.plugin.uninstall"), Messages.getQuestionIcon()) == Messages.YES);