From f44256ecc562b7e54e9e49483464ca6306fba3f8 Mon Sep 17 00:00:00 2001
From: Konstantin Bulenkov <kb@jetbrains.com>
Date: Tue, 10 May 2016 14:14:32 +0200
Subject: [PATCH] move recent projects duplicates calculation from EDT

---
 .../ide/RecentProjectsManagerBase.java        | 30 ++++++++++++++-----
 1 file changed, 23 insertions(+), 7 deletions(-)

diff --git a/platform/platform-impl/src/com/intellij/ide/RecentProjectsManagerBase.java b/platform/platform-impl/src/com/intellij/ide/RecentProjectsManagerBase.java
index 49fbce020330..7d2ceea1b4b5 100644
--- a/platform/platform-impl/src/com/intellij/ide/RecentProjectsManagerBase.java
+++ b/platform/platform-impl/src/com/intellij/ide/RecentProjectsManagerBase.java
@@ -99,6 +99,8 @@ public abstract class RecentProjectsManagerBase extends RecentProjectsManager im
   private State myState = new State();
 
   private final Map<String, String> myNameCache = Collections.synchronizedMap(new THashMap<String, String>());
+  private Set<String> myDuplicatesCache = null;
+  private boolean isDuplicatesCacheUpdating = false;
 
   protected RecentProjectsManagerBase(@NotNull MessageBus messageBus) {
     MessageBusConnection connection = messageBus.connect();
@@ -328,14 +330,28 @@ public abstract class RecentProjectsManagerBase extends RecentProjectsManager im
   }
 
   private Set<String> getDuplicateProjectNames(Set<String> openedPaths, Set<String> recentPaths) {
-    Set<String> names = ContainerUtil.newHashSet();
-    Set<String> duplicates = ContainerUtil.newHashSet();
-    for (String path : ContainerUtil.concat(openedPaths, recentPaths)) {
-      if (!names.add(getProjectName(path))) {
-        duplicates.add(path);
-      }
+    if (myDuplicatesCache != null) {
+      return myDuplicatesCache;
     }
-    return duplicates;
+
+    if (!isDuplicatesCacheUpdating) {
+      isDuplicatesCacheUpdating = true; //assuming that this check happens only on EDT. So, no synchronised block or double-checked locking needed
+      ApplicationManager.getApplication().executeOnPooledThread(new Runnable() {
+        @Override
+        public void run() {
+          Set<String> names = ContainerUtil.newHashSet();
+          final HashSet<String> duplicates = ContainerUtil.newHashSet();
+          for (String path : ContainerUtil.concat(openedPaths, recentPaths)) {
+            if (!names.add(getProjectName(path))) {
+              duplicates.add(path);
+            }
+          }
+          myDuplicatesCache = duplicates;
+          isDuplicatesCacheUpdating = false;
+        }
+      });
+    }
+    return ContainerUtil.newHashSet();
   }
 
   @Override