slight tweak to completion logic, bringing code in line with intention

SethTisue · lrytz · web-flow · commit f125f82a14b9 · 2023-05-25T10:13:43.000-10:00
Co-authored-by: Lukas Rytz &lt;lukas.rytz@gmail.com&gt;
diff --git a/src/interactive/scala/tools/nsc/interactive/Global.scala b/src/interactive/scala/tools/nsc/interactive/Global.scala
@@ -1190,7 +1190,7 @@ class Global(settings: Settings, _reporter: Reporter, projectName: String = "")
             symbol.name.isTermName == name.isTermName || // Keep names of the same type
             name.isTypeName && isStable // Completing a type: keep stable terms (paths)
         }
-        !isJunk && member.accessible && !symbol.isConstructor && (name.isEmpty || matcher(member.sym.name) && nameTypeOk)
+        !isJunk && member.accessible && !symbol.isConstructor && (name.isEmpty || matcher(member.sym.name)) && nameTypeOk
 
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -1190,7 +1190,7 @@ class Global(settings: Settings, _reporter: Reporter, projectName: String = "")`
`1190`	`1190`	`symbol.name.isTermName == name.isTermName \|\| // Keep names of the same type`
`1191`	`1191`	`name.isTypeName && isStable // Completing a type: keep stable terms (paths)`
`1192`	`1192`	`}`
`1193`		`- !isJunk && member.accessible && !symbol.isConstructor && (name.isEmpty \|\| matcher(member.sym.name) && nameTypeOk)`
	`1193`	`+ !isJunk && member.accessible && !symbol.isConstructor && (name.isEmpty \|\| matcher(member.sym.name)) && nameTypeOk`
`1194`	`1194`
`1195`	`1195`	`}`
`1196`	`1196`	`}`