From 7f7348b46efcb6eda7946c5eab6bd8d1f6dea0d9 Mon Sep 17 00:00:00 2001
From: Dmitry Jemerov <yole@jetbrains.com>
Date: Thu, 9 Jan 2020 18:57:41 +0100
Subject: [PATCH] Add clarifying comment (IDEA-CR-57121)

GitOrigin-RevId: 75ee7d892786a41ddb47889da242f0dcfe3262e2
---
 .../intellij/psi/impl/file/impl/ResolveScopeManagerImpl.java    | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/platform/indexing-impl/src/com/intellij/psi/impl/file/impl/ResolveScopeManagerImpl.java b/platform/indexing-impl/src/com/intellij/psi/impl/file/impl/ResolveScopeManagerImpl.java
index e108a38639cf..b8cfa6ec7ce1 100644
--- a/platform/indexing-impl/src/com/intellij/psi/impl/file/impl/ResolveScopeManagerImpl.java
+++ b/platform/indexing-impl/src/com/intellij/psi/impl/file/impl/ResolveScopeManagerImpl.java
@@ -56,6 +56,8 @@ public final class ResolveScopeManagerImpl extends ResolveScopeManager {
       ContainerUtil::createConcurrentWeakKeySoftValueMap);
 
     ((PsiManagerImpl)myManager).registerRunnableToRunOnChange(myDefaultResolveScopesCache::clear);
+    // Make it explicit that registering and removing ResolveScopeProviders needs to clear the resolve scope cache
+    // (even though normally registerRunnableToRunOnChange would be enough to clear the cache)
     ResolveScopeProvider.EP_NAME.addExtensionPointListener(() -> myDefaultResolveScopesCache.clear(), project);
   }