MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2025-12-16 22:51:17 +07:00
Code Issues Packages Projects Releases Wiki Activity

KMA-432 Provide mechanism that inheritor searchers can use to avoid unnecessary work for participating queries

Browse Source 
GitOrigin-RevId: 84cb945d4f6e979de56adcb50e3642bda80b9ab6
This commit is contained in:
Florian Kistner
2022-05-30 20:26:46 +02:00
committed by intellij-monorepo-bot
parent aebcdf66d6
commit 6b13ec47a5
3 changed files with 40 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
java/java-indexing-impl/src/com/intellij/psi/impl/search/JavaClassInheritorsSearcher.java
View File

@@ -2,6 +2,7 @@
package com.intellij.psi.impl.search;
import com.intellij.java.indexing.JavaIndexingBundle;
import com.intellij.lang.Language;
import com.intellij.openapi.application.ApplicationManager;
import com.intellij.openapi.application.QueryExecutorBase;
import com.intellij.openapi.application.ReadAction;
@@ -78,7 +79,7 @@ public class JavaClassInheritorsSearcher extends QueryExecutorBase<PsiClass, Cla
return;
}
Iterable<PsiClass> cached = getOrComputeSubClasses(project, baseClass, searchScope, parameters.isIncludeAnonymous());
Iterable<PsiClass> cached = getOrComputeSubClasses(project, baseClass, searchScope, parameters);
for (final PsiClass subClass : cached) {
ProgressManager.checkCanceled();
@@ -97,9 +98,11 @@ public class JavaClassInheritorsSearcher extends QueryExecutorBase<PsiClass, Cla
private static Iterable<@NotNull PsiClass> getOrComputeSubClasses(@NotNull Project project,
@NotNull PsiClass baseClass,
@NotNull SearchScope searchScopeForNonPhysical,
boolean includeAnonymous) {
@NotNull ClassInheritorsSearch.SearchParameters parameters) {
HighlightingCaches caches = HighlightingCaches.getInstance(project);
ConcurrentMap<PsiClass, Iterable<PsiClass>> map = includeAnonymous ? caches.ALL_SUB_CLASSES : caches.ALL_SUB_CLASSES_NO_ANONYMOUS;
ConcurrentMap<PsiClass, Iterable<PsiClass>> map = parameters.isIncludeAnonymous()
? caches.ALL_SUB_CLASSES
: caches.ALL_SUB_CLASSES_NO_ANONYMOUS;
Iterable<PsiClass> cached = map.get(baseClass);
if (cached == null) {
// returns lazy collection of subclasses. Each call to next() leads to calculation of next batch of subclasses.
@@ -110,14 +113,20 @@ public class JavaClassInheritorsSearcher extends QueryExecutorBase<PsiClass, Cla
// for non-physical elements ignore the cache completely because non-physical elements created so often/unpredictably so I can't figure out when to clear caches in this case
boolean isPhysical = ReadAction.compute(baseClass::isPhysical);
SearchScope scopeToUse = isPhysical ? GlobalSearchScope.allScope(project) : searchScopeForNonPhysical;
LazyConcurrentCollection.MoreElementsGenerator<PsiAnchor, PsiClass> generator = (candidate, processor) ->
DirectClassInheritorsSearch.search(candidate, scopeToUse, includeAnonymous).allowParallelProcessing().forEach(subClass -> {
ProgressManager.checkCanceled();
@NotNull PsiAnchor pointer = ReadAction.compute(() -> PsiAnchor.create(subClass));
// append found result to subClasses as early as possible to allow other waiting threads to continue
processor.accept(pointer);
return true;
});
LazyConcurrentCollection.MoreElementsGenerator<PsiAnchor, PsiClass> generator = (candidate, processor) -> DirectClassInheritorsSearch
.search(new DirectClassInheritorsSearch.SearchParameters(candidate, scopeToUse, parameters.isIncludeAnonymous(), true) {
@Override
public boolean shouldSearchInLanguage(@NotNull Language language) {
return parameters.shouldSearchInLanguage(language);
}
})
.allowParallelProcessing().forEach(subClass -> {
ProgressManager.checkCanceled();
@NotNull PsiAnchor pointer = ReadAction.compute(() -> PsiAnchor.create(subClass));
// append found result to subClasses as early as possible to allow other waiting threads to continue
processor.accept(pointer);
return true;
});
PsiAnchor seed = ReadAction.compute(() -> PsiAnchor.create(baseClass));
// lazy collection: store underlying queue as PsiAnchors, generate new elements by running direct inheritors