From 05fae1d4dbca63cdedfac262e92379a73f45f72e Mon Sep 17 00:00:00 2001
From: Valentin Kipyatkov <valentin@jetbrains.com>
Date: Tue, 30 Aug 2016 20:27:44 +0300
Subject: [PATCH] Optimization: don't search named arguments

---
 .../idea/search/ideaExtensions/KotlinReferencesSearcher.kt | 7 ++++---
 .../search/usagesSearch/destructuringDeclarationUsages.kt  | 6 +++++-
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/search/ideaExtensions/KotlinReferencesSearcher.kt b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/search/ideaExtensions/KotlinReferencesSearcher.kt
index d9743fb9e2c..0725ce85ee1 100644
--- a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/search/ideaExtensions/KotlinReferencesSearcher.kt
+++ b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/search/ideaExtensions/KotlinReferencesSearcher.kt
@@ -52,11 +52,12 @@ data class KotlinReferencesSearchOptions(val acceptCallableOverrides: Boolean =
                                          val acceptOverloads: Boolean = false,
                                          val acceptExtensionsOfDeclarationClass: Boolean = false,
                                          val acceptCompanionObjectMembers: Boolean = false,
-                                         val searchForComponentConventions: Boolean = true) {
+                                         val searchForComponentConventions: Boolean = true,
+                                         val searchNamedArguments: Boolean = true) {
     fun anyEnabled(): Boolean = acceptCallableOverrides || acceptOverloads || acceptExtensionsOfDeclarationClass
 
     companion object {
-        val Empty = KotlinReferencesSearchOptions(false, false, false)
+        val Empty = KotlinReferencesSearchOptions()
     }
 }
 
@@ -111,7 +112,7 @@ class KotlinReferencesSearcher : QueryExecutorBase<PsiReference, ReferencesSearc
                                                  resultProcessor)
         }
 
-        if (unwrappedElement is KtParameter) {
+        if (unwrappedElement is KtParameter && kotlinOptions.searchNamedArguments) {
             runReadAction { searchNamedArguments(unwrappedElement, queryParameters) }
         }
 
diff --git a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/search/usagesSearch/destructuringDeclarationUsages.kt b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/search/usagesSearch/destructuringDeclarationUsages.kt
index 38e797734b7..09dad6941c9 100644
--- a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/search/usagesSearch/destructuringDeclarationUsages.kt
+++ b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/search/usagesSearch/destructuringDeclarationUsages.kt
@@ -40,6 +40,8 @@ import org.jetbrains.kotlin.idea.caches.resolve.resolveToDescriptorIfAny
 import org.jetbrains.kotlin.idea.codeInsight.DescriptorToSourceUtilsIde
 import org.jetbrains.kotlin.idea.refactoring.fqName.getKotlinFqName
 import org.jetbrains.kotlin.idea.references.KtDestructuringDeclarationReference
+import org.jetbrains.kotlin.idea.search.ideaExtensions.KotlinReferencesSearchOptions
+import org.jetbrains.kotlin.idea.search.ideaExtensions.KotlinReferencesSearchParameters
 import org.jetbrains.kotlin.idea.search.ideaExtensions.KotlinRequestResultProcessor
 import org.jetbrains.kotlin.idea.search.restrictToKotlinSources
 import org.jetbrains.kotlin.idea.search.usagesSearch.DestructuringDeclarationUsageSearch.*
@@ -260,7 +262,9 @@ private class Processor(
                 //TODO: what about Scala and other JVM-languages?
                 val scope = GlobalSearchScope.projectScope(project).restrictToKotlinSources()
                 testLog?.add("Searched references to ${declaration.logPresentation()} in Kotlin files")
-                ReferencesSearch.search(declaration, scope).forEach { reference ->
+                val searchParameters = KotlinReferencesSearchParameters(
+                        declaration, scope, kotlinOptions = KotlinReferencesSearchOptions(searchNamedArguments = false))
+                ReferencesSearch.search(searchParameters).forEach { reference ->
                     when (kind) {
                         CallableToProcessKind.HAS_DATA_CLASS_TYPE -> {
                             if (reference is KtDestructuringDeclarationReference) {