Review changes update

scalameta · Feb 18, 2020 · db2d22b · db2d22b
1 parent 37012e4
commit db2d22b
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/metals/src/main/scala/scala/meta/internal/metals/ReferenceProvider.scala b/metals/src/main/scala/scala/meta/internal/metals/ReferenceProvider.scala
@@ -94,9 +94,10 @@ final class ReferenceProvider(
         maybeOccurrence match {
           case Some(occurrence) =>
             val symbolName = occurrence.symbol.desc.name.value
-            if (ReferenceProvider.methodsSearchedWithoutInheritance.contains(
-                symbolName
-              )) {
+            val shouldIncludeInheritance =
+              ReferenceProvider.methodsSearchedWithoutInheritance.contains(symbolName)
+
+            if (shouldIncludeInheritance) {
               currentSymbolReferences(
                 filePosition,
                 includeDeclaration
@@ -128,12 +129,12 @@ final class ReferenceProvider(
   ): Seq[Location] = {
     val parentSymbols = implementation
       .topMethodParents(doc, symbolOccurrence.symbol)
-    val txtParams: Seq[FilePosition] = {
+    val parentPositions: Seq[FilePosition] = {
       if (parentSymbols.isEmpty) List(filePosition)
       else parentSymbols.map(locationToFilePosition)
     }
     val isLocal = symbolOccurrence.symbol.isLocal
-    val currentReferences = txtParams
+    val currentReferences = parentPositions
       .flatMap(
         currentSymbolReferences(
           _,
@@ -150,7 +151,7 @@ final class ReferenceProvider(
           .filter(_.getUri.isScalaFilename)
       else parentSymbols
     }
-    val implReferences = txtParams.flatMap(
+    val implReferences = parentPositions.flatMap(
       implementations(_, !symbolOccurrence.symbol.desc.isType)
     )