Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 0 additions & 33 deletions compiler/src/dotty/tools/dotc/interactive/Interactive.scala
Original file line number Diff line number Diff line change
Expand Up @@ -94,39 +94,6 @@ object Interactive {
private def safely[T](op: => List[T]): List[T] =
try op catch { case ex: TypeError => Nil }

/** Get possible completions from tree at `pos`
*
* @return offset and list of symbols for possible completions
*/
// deprecated
// FIXME: Remove this method
def completions(trees: List[SourceTree], pos: SourcePosition)(implicit ctx: Context): (Int, List[Symbol]) = {
val path = pathTo(trees, pos)
val boundary = enclosingDefinitionInPath(path).symbol

// FIXME: Get all declarations available in the current scope, not just
// those from the enclosing class
def scopeCompletions: List[Symbol] =
boundary.enclosingClass match {
case csym: ClassSymbol =>
val classRef = csym.classInfo.appliedRef
completions(classRef, boundary)
case _ =>
Nil
}

path.headOption.map {
case sel @ Select(qual, name) =>
// When completing "`a.foo`, return the members of `a`
(sel.pos.point, completions(qual.tpe, boundary))
case id @ Ident(name) =>
(id.pos.point, scopeCompletions)
case _ =>
(0, scopeCompletions)
}
.getOrElse((0, Nil))
}

/** Get possible completions from tree at `pos`
*
* @return offset and list of symbols for possible completions
Expand Down
7 changes: 4 additions & 3 deletions compiler/src/dotty/tools/repl/ReplDriver.scala
Original file line number Diff line number Diff line change
Expand Up @@ -169,17 +169,18 @@ class ReplDriver(settings: Array[String],
compiler
.typeCheck(expr, errorsAllowed = true)
.map { tree =>
implicit val ctx: Context = state.run.runContext
val file = new dotc.util.SourceFile("compl", expr)
val unit = new CompilationUnit(file)
unit.tpdTree = tree
implicit val ctx: Context = state.run.runContext.fresh.setCompilationUnit(unit)
val srcPos = dotc.util.SourcePosition(file, Position(cursor))
val (startOffset, completions) = Interactive.completions(SourceTree(tree, file) :: Nil, srcPos)(ctx)
val (startOffset, completions) = Interactive.completions(srcPos)
val query =
if (startOffset < cursor) expr.substring(startOffset, cursor) else ""

def filterCompletions(name: String) =
(query == "." || name.startsWith(query)) && name != query


Completions(
Math.min(startOffset, cursor) + { if (query == ".") 1 else 0 },
completions.map(_.name.show).distinct.filter(filterCompletions),
Expand Down
3 changes: 2 additions & 1 deletion compiler/test/dotty/tools/repl/TabcompleteTests.scala
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,8 @@ class TabcompleteTests extends ReplTest {

@Test def i3309: Unit =
fromInitialState { implicit s =>
List("\"", "#", ")", "=", "'", "¨", "£", ".", ":", ",", ";", "@", "}", "[", "]")
// TODO: add back '.', once #4397 is fixed
List("\"", ")", "'", "¨", "£", ":", ",", ";", "@", "}", "[", "]")
.foreach(src => assertTrue(tabComplete(src).suggestions.isEmpty))
}
}