From 1729d9de4dffb8fb70c087faa8a7affe5716cccf Mon Sep 17 00:00:00 2001
From: Kirill Kalishev <kirill.kalishev@jetbrains.com>
Date: Mon, 5 Oct 2009 07:43:54 +0400
Subject: [PATCH] TreeUi: used sets instead of maps where possible

---
 .../intellij/ide/util/treeView/AbstractTreeUi.java | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/platform/platform-api/src/com/intellij/ide/util/treeView/AbstractTreeUi.java b/platform/platform-api/src/com/intellij/ide/util/treeView/AbstractTreeUi.java
index 13f6ccf4d019..2720e768d6c7 100644
--- a/platform/platform-api/src/com/intellij/ide/util/treeView/AbstractTreeUi.java
+++ b/platform/platform-api/src/com/intellij/ide/util/treeView/AbstractTreeUi.java
@@ -66,7 +66,7 @@ class AbstractTreeUi {
   private MySelectionListener mySelectionListener;
 
   private WorkerThread myWorker = null;
-  private final ArrayList<Runnable> myActiveWorkerTasks = new ArrayList<Runnable>();
+  private final Set<Runnable> myActiveWorkerTasks = new HashSet<Runnable>();
 
   private ProgressIndicator myProgress;
   private static final int WAIT_CURSOR_DELAY = 100;
@@ -77,15 +77,15 @@ class AbstractTreeUi {
   private boolean myWasEverShown;
   private boolean myUpdateIfInactive;
 
-  private final List<Object> myLoadingParents = new ArrayList<Object>();
+  private final Set<Object> myLoadingParents = new HashSet<Object>();
   private final Map<Object, List<NodeAction>> myNodeChildrenActions = new HashMap<Object, List<NodeAction>>();
 
   private long myClearOnHideDelay = -1;
   private ScheduledExecutorService ourClearanceService;
   private final Map<AbstractTreeUi, Long> ourUi2Countdown = Collections.synchronizedMap(new WeakHashMap<AbstractTreeUi, Long>());
 
-  private final List<Runnable> myDeferredSelections = new ArrayList<Runnable>();
-  private final List<Runnable> myDeferredExpansions = new ArrayList<Runnable>();
+  private final Set<Runnable> myDeferredSelections = new HashSet<Runnable>();
+  private final Set<Runnable> myDeferredExpansions = new HashSet<Runnable>();
 
   private boolean myCanProcessDeferredSelections;
 
@@ -103,8 +103,8 @@ class AbstractTreeUi {
 
   private boolean myYeildingNow;
 
-  private final List<DefaultMutableTreeNode> myPendingNodeActions = new ArrayList<DefaultMutableTreeNode>();
-  private final List<Runnable> myYeildingDoneRunnables = new ArrayList<Runnable>();
+  private final Set<DefaultMutableTreeNode> myPendingNodeActions = new HashSet<DefaultMutableTreeNode>();
+  private final Set<Runnable> myYeildingDoneRunnables = new HashSet<Runnable>();
 
   private final Alarm myBusyAlarm = new Alarm();
   private final Runnable myWaiterForReady = new Runnable() {
@@ -605,7 +605,7 @@ class AbstractTreeUi {
     processDeferredActions(myDeferredExpansions);
   }
 
-  private void processDeferredActions(List<Runnable> actions) {
+  private void processDeferredActions(Set<Runnable> actions) {
     final Runnable[] runnables = actions.toArray(new Runnable[actions.size()]);
     actions.clear();
     for (Runnable runnable : runnables) {