Exclude primary constructors in namedTrees

We never want to match them, so it's easier to exclude them here rather
than always add a check in the predicate function.

Author: Duhemm

compiler/src/dotty/tools/dotc/interactive/Interactive.scala

@@ -338,6 +338,7 @@ object Interactive {
               val tree = utree.asInstanceOf[tpd.NameTree]
               if (tree.symbol.exists
                    && !tree.symbol.is(Synthetic)
+                   && !tree.symbol.isPrimaryConstructor
                    && tree.pos.exists
                    && !tree.pos.isZeroExtent
                    && (include.isReferences || isDefinition(tree))
@@ -373,8 +374,7 @@ object Interactive {
                        )(implicit ctx: Context): List[SourceTree] = {
     val linkedSym = symbol.linkedClass
     val fullPredicate: NameTree => Boolean = tree =>
-      (  !tree.symbol.isPrimaryConstructor
-      && (includes.isDefinitions || !Interactive.isDefinition(tree))
+      (  (includes.isDefinitions || !Interactive.isDefinition(tree))
       && (  Interactive.matchSymbol(tree, symbol, includes)
          || ( includes.isLinkedClass
             && linkedSym.exists

language-server/src/dotty/tools/languageserver/DottyLanguageServer.scala

@@ -457,10 +457,7 @@ class DottyLanguageServer extends LanguageServer
     implicit val ctx = driver.currentCtx
 
     val uriTrees = driver.openedTrees(uri)
-    val predicate = (tree: NameTree) => {
-      val sym = tree.symbol
-      !sym.isLocal && !sym.isPrimaryConstructor
-    }
+    val predicate = (tree: NameTree) => !tree.symbol.isLocal
 
     val defs = Interactive.namedTrees(uriTrees, Include.empty, predicate)
     (for {
@@ -473,7 +470,7 @@ class DottyLanguageServer extends LanguageServer
     val query = params.getQuery
     def predicate(implicit ctx: Context): NameTree => Boolean = { tree =>
       val sym = tree.symbol
-      !sym.isLocal && !sym.isPrimaryConstructor && tree.name.toString.contains(query)
+      !sym.isLocal && tree.name.toString.contains(query)
     }
 
     drivers.values.toList.flatMap { driver =>