From 6d5fde94a17aab2a5b23fa9fa404df86d198125f Mon Sep 17 00:00:00 2001
From: Constantine Plotnikov <Constantine.Plotnikov@jetbrains.com>
Date: Tue, 2 Jun 2009 17:31:06 +0400
Subject: [PATCH] git4idea: optimized root tracker to preform fs checks only if
 there are configured git roots.

---
 .../git4idea/src/git4idea/vfs/GitRootTracker.java  | 38 ++++++++++++++++++++--
 1 file changed, 35 insertions(+), 3 deletions(-)

diff --git a/plugins/git4idea/src/git4idea/vfs/GitRootTracker.java b/plugins/git4idea/src/git4idea/vfs/GitRootTracker.java
index 764a82b219..47ef5aab68 100644
--- a/plugins/git4idea/src/git4idea/vfs/GitRootTracker.java
+++ b/plugins/git4idea/src/git4idea/vfs/GitRootTracker.java
@@ -66,6 +66,10 @@ public class GitRootTracker implements VcsListener {
    */
   private final AtomicBoolean myRootsInvalidated = new AtomicBoolean(true);
   /**
+   * If true, there are some configured git roots, or listener has never been run yet
+   */
+  private final AtomicBoolean myHasGitRoots = new AtomicBoolean(true);
+  /**
    * If true, the notification is currently active and has not been dismissed yet.
    */
   private final AtomicBoolean myNotificationPosted = new AtomicBoolean(false);
@@ -110,7 +114,7 @@ public class GitRootTracker implements VcsListener {
         if (!myRootsInvalidated.compareAndSet(true, false)) {
           return;
         }
-        directoryMappingChanged();
+        checkRoots(false);
       }
     };
     CommandProcessor.getInstance().addCommandListener(myCommandListener);
@@ -123,11 +127,11 @@ public class GitRootTracker implements VcsListener {
         if (!myRootsInvalidated.compareAndSet(true, false)) {
           return;
         }
-        directoryMappingChanged();
+        checkRoots(false);
       }
     };
     fileManager.addVirtualFileManagerListener(myVirtualFileManagerListener);
-    directoryMappingChanged();
+    checkRoots(true);
   }
 
   /**
@@ -148,6 +152,18 @@ public class GitRootTracker implements VcsListener {
     if (myProject.isDisposed()) {
       return;
     }
+    checkRoots(true);
+  }
+
+  /**
+   * Check roots for changes.
+   *
+   * @param rootsChanged
+   */
+  private void checkRoots(boolean rootsChanged) {
+    if (!rootsChanged && !myHasGitRoots.get()) {
+      return;
+    }
     ApplicationManager.getApplication().runReadAction(new Runnable() {
       public void run() {
         boolean hasInvalidRoots = false;
@@ -172,8 +188,12 @@ public class GitRootTracker implements VcsListener {
           }
         }
         if (!hasInvalidRoots && rootSet.isEmpty()) {
+          myHasGitRoots.set(false);
           return;
         }
+        else {
+          myHasGitRoots.set(true);
+        }
         if (!hasInvalidRoots) {
           // check if roots have a problem
           loop:
@@ -417,6 +437,9 @@ public class GitRootTracker implements VcsListener {
      */
     @Override
     public void fileCreated(VirtualFileEvent event) {
+      if (!myHasGitRoots.get()) {
+        return;
+      }
       if (hasGitRepositories(event.getFile())) {
         invalidate();
       }
@@ -428,6 +451,9 @@ public class GitRootTracker implements VcsListener {
      */
     @Override
     public void beforeFileDeletion(VirtualFileEvent event) {
+      if (!myHasGitRoots.get()) {
+        return;
+      }
       if (hasGitRepositories(event.getFile())) {
         invalidate();
       }
@@ -438,6 +464,9 @@ public class GitRootTracker implements VcsListener {
      */
     @Override
     public void fileMoved(VirtualFileMoveEvent event) {
+      if (!myHasGitRoots.get()) {
+        return;
+      }
       if (hasGitRepositories(event.getFile())) {
         invalidate();
       }
@@ -448,6 +477,9 @@ public class GitRootTracker implements VcsListener {
      */
     @Override
     public void fileCopied(VirtualFileCopyEvent event) {
+      if (!myHasGitRoots.get()) {
+        return;
+      }
       if (hasGitRepositories(event.getFile())) {
         invalidate();
       }
-- 
2.11.4.GIT