From 2ef9634091a42e88170b695c9aa5b60958a2e661 Mon Sep 17 00:00:00 2001 From: Evgeny Gerashchenko Date: Wed, 21 Jan 2015 16:40:10 +0300 Subject: [PATCH] Clarified weird NPE: EA-64049 --- .../kotlin/idea/inspections/UnusedSymbolInspection.kt | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/idea/src/org/jetbrains/kotlin/idea/inspections/UnusedSymbolInspection.kt b/idea/src/org/jetbrains/kotlin/idea/inspections/UnusedSymbolInspection.kt index 8e1b16fa291..c09a85dd511 100644 --- a/idea/src/org/jetbrains/kotlin/idea/inspections/UnusedSymbolInspection.kt +++ b/idea/src/org/jetbrains/kotlin/idea/inspections/UnusedSymbolInspection.kt @@ -58,6 +58,7 @@ import com.intellij.codeInspection.ProblemDescriptor import com.intellij.codeInsight.daemon.QuickFixBundle import com.intellij.codeInsight.FileModificationService import com.intellij.refactoring.safeDelete.SafeDeleteHandler +import org.jetbrains.kotlin.psi.JetPsiUtil public class UnusedSymbolInspection : AbstractKotlinInspection() { private val javaInspection = UnusedDeclarationInspection() @@ -222,6 +223,10 @@ public class UnusedSymbolInspection : AbstractKotlinInspection() { val request = searchHelper.newRequest(UsagesSearchTarget(declaration, useScope)) val query = UsagesSearch.search(request) - return !query.forEach(Processor { declaration.isAncestor(it.getElement()) }) + return !query.forEach(Processor { + assert(it != null, { "Found reference is null, was looking for: " + JetPsiUtil.getElementTextWithContext(declaration) + + " findAll(): " + query.findAll().map { it?.getElement()?.let{ JetPsiUtil.getElementTextWithContext(it) } } }) + declaration.isAncestor(it.getElement()) + }) } }