Skip to content

Commit

Permalink
Add syntax for qualified types
Browse files Browse the repository at this point in the history
Co-Authored-By: Quentin Bernet <[email protected]>
  • Loading branch information
mbovel and Sporarum committed Nov 11, 2024
1 parent d78516a commit d5021f8
Show file tree
Hide file tree
Showing 14 changed files with 385 additions and 7 deletions.
28 changes: 26 additions & 2 deletions compiler/src/dotty/tools/dotc/ast/Desugar.scala
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import Symbols.*, StdNames.*, Trees.*, ContextOps.*
import Decorators.*
import Annotations.Annotation
import NameKinds.{UniqueName, ContextBoundParamName, ContextFunctionParamName, DefaultGetterName, WildcardParamName}
import typer.{Namer, Checking}
import typer.{Namer, Checking, ErrorReporting}
import util.{Property, SourceFile, SourcePosition, SrcPos, Chars}
import config.{Feature, Config}
import config.Feature.{sourceVersion, migrateTo3, enabled, betterForsEnabled}
Expand Down Expand Up @@ -199,9 +199,10 @@ object desugar {
def valDef(vdef0: ValDef)(using Context): Tree =
val vdef @ ValDef(_, tpt, rhs) = vdef0
val valName = normalizeName(vdef, tpt).asTermName
val tpt1 = qualifiedType(tpt, valName)
var mods1 = vdef.mods

val vdef1 = cpy.ValDef(vdef)(name = valName).withMods(mods1)
val vdef1 = cpy.ValDef(vdef)(name = valName, tpt = tpt1).withMods(mods1)

if isSetterNeeded(vdef) then
val setterParam = makeSyntheticParameter(tpt = SetterParamTree().watching(vdef))
Expand Down Expand Up @@ -2158,6 +2159,10 @@ object desugar {
case PatDef(mods, pats, tpt, rhs) =>
val pats1 = if (tpt.isEmpty) pats else pats map (Typed(_, tpt))
flatTree(pats1 map (makePatDef(tree, mods, _, rhs)))
case QualifiedTypeTree(parent, None, qualifier) =>
ErrorReporting.errorTree(parent, em"missing parameter name in qualified type", tree.srcPos)
case QualifiedTypeTree(parent, Some(paramName), qualifier) =>
qualifiedType(parent, paramName, qualifier, tree.span)
case ext: ExtMethods =>
Block(List(ext), syntheticUnitLiteral.withSpan(ext.span))
case f: FunctionWithMods if f.hasErasedParams => makeFunctionWithValDefs(f, pt)
Expand Down Expand Up @@ -2336,4 +2341,23 @@ object desugar {
collect(tree)
buf.toList
}

/** If `tree` is a `QualifiedTypeTree`, then desugars it using `paramName` as
* the qualified parameter name. Otherwise, returns `tree` unchanged.
*/
def qualifiedType(tree: Tree, paramName: TermName)(using Context): Tree = tree match
case QualifiedTypeTree(parent, None, qualifier) => qualifiedType(parent, paramName, qualifier, tree.span)
case _ => tree

/** Returns the annotated type used to represent the qualified type with the
* given components:
* `parent @qualified[parent]((paramName: parent) => qualifier)`.
*/
def qualifiedType(parent: Tree, paramName: TermName, qualifier: Tree, span: Span)(using Context): Tree =
val param = makeParameter(paramName, parent, EmptyModifiers) // paramName: parent
val predicate = WildcardFunction(List(param), qualifier) // (paramName: parent) => qualifier
val qualifiedAnnot = scalaAnnotationDot(nme.qualified)
val annot = Apply(TypeApply(qualifiedAnnot, List(parent)), predicate).withSpan(span) // @qualified[parent](predicate)
Annotated(parent, annot).withSpan(span) // parent @qualified[parent](predicate)

}
15 changes: 15 additions & 0 deletions compiler/src/dotty/tools/dotc/ast/untpd.scala
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,13 @@ object untpd extends Trees.Instance[Untyped] with UntypedTreeInfo {
*/
case class CapturesAndResult(refs: List[Tree], parent: Tree)(implicit @constructorOnly src: SourceFile) extends TypTree

/** `{ x: parent with qualifier }` if `paramName == Some(x)`,
* `parent with qualifier` otherwise.
*
* Only relevant under `qualifiedTypes`.
*/
case class QualifiedTypeTree(parent: Tree, paramName: Option[TermName], qualifier: Tree)(implicit @constructorOnly src: SourceFile) extends TypTree

/** A type tree appearing somewhere in the untyped DefDef of a lambda, it will be typed using `tpFun`.
*
* @param isResult Is this the result type of the lambda? This is handled specially in `Namer#valOrDefDefSig`.
Expand Down Expand Up @@ -703,6 +710,10 @@ object untpd extends Trees.Instance[Untyped] with UntypedTreeInfo {
case tree: CapturesAndResult if (refs eq tree.refs) && (parent eq tree.parent) => tree
case _ => finalize(tree, untpd.CapturesAndResult(refs, parent))

def QualifiedTypeTree(tree: Tree)(parent: Tree, paramName: Option[TermName], qualifier: Tree)(using Context): Tree = tree match
case tree: QualifiedTypeTree if (parent eq tree.parent) && (paramName eq tree.paramName) && (qualifier eq tree.qualifier) => tree
case _ => finalize(tree, untpd.QualifiedTypeTree(parent, paramName, qualifier)(tree.source))

def TypedSplice(tree: Tree)(splice: tpd.Tree)(using Context): ProxyTree = tree match {
case tree: TypedSplice if splice `eq` tree.splice => tree
case _ => finalize(tree, untpd.TypedSplice(splice)(using ctx))
Expand Down Expand Up @@ -766,6 +777,8 @@ object untpd extends Trees.Instance[Untyped] with UntypedTreeInfo {
cpy.MacroTree(tree)(transform(expr))
case CapturesAndResult(refs, parent) =>
cpy.CapturesAndResult(tree)(transform(refs), transform(parent))
case QualifiedTypeTree(parent, paramName, qualifier) =>
cpy.QualifiedTypeTree(tree)(transform(parent), paramName, transform(qualifier))
case _ =>
super.transformMoreCases(tree)
}
Expand Down Expand Up @@ -825,6 +838,8 @@ object untpd extends Trees.Instance[Untyped] with UntypedTreeInfo {
this(x, expr)
case CapturesAndResult(refs, parent) =>
this(this(x, refs), parent)
case QualifiedTypeTree(parent, paramName, qualifier) =>
this(this(x, parent), qualifier)
case _ =>
super.foldMoreCases(x, tree)
}
Expand Down
6 changes: 6 additions & 0 deletions compiler/src/dotty/tools/dotc/config/Feature.scala
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ object Feature:
val clauseInterleaving = experimental("clauseInterleaving")
val pureFunctions = experimental("pureFunctions")
val captureChecking = experimental("captureChecking")
val qualifiedTypes = experimental("qualifiedTypes")
val into = experimental("into")
val modularity = experimental("modularity")
val betterMatchTypeExtractors = experimental("betterMatchTypeExtractors")
Expand Down Expand Up @@ -64,6 +65,7 @@ object Feature:
(clauseInterleaving, "Enable clause interleaving"),
(pureFunctions, "Enable pure functions for capture checking"),
(captureChecking, "Enable experimental capture checking"),
(qualifiedTypes, "Enable experimental qualified types"),
(into, "Allow into modifier on parameter types"),
(modularity, "Enable experimental modularity features"),
(betterMatchTypeExtractors, "Enable better match type extractors"),
Expand Down Expand Up @@ -156,6 +158,10 @@ object Feature:
if ctx.run != null then ctx.run.nn.ccEnabledSomewhere
else enabledBySetting(captureChecking)

/** Is qualifiedTypes enabled for this compilation unit? */
def qualifiedTypesEnabled(using Context) =
enabledBySetting(qualifiedTypes)

def sourceVersionSetting(using Context): SourceVersion =
SourceVersion.valueOf(ctx.settings.source.value)

Expand Down
1 change: 1 addition & 0 deletions compiler/src/dotty/tools/dotc/core/StdNames.scala
Original file line number Diff line number Diff line change
Expand Up @@ -585,6 +585,7 @@ object StdNames {
val productElementName: N = "productElementName"
val productIterator: N = "productIterator"
val productPrefix: N = "productPrefix"
val qualified : N = "qualified"
val quotes : N = "quotes"
val raw_ : N = "raw"
val refl: N = "refl"
Expand Down
42 changes: 38 additions & 4 deletions compiler/src/dotty/tools/dotc/parsing/Parsers.scala
Original file line number Diff line number Diff line change
Expand Up @@ -445,6 +445,13 @@ object Parsers {
finally inMatchPattern = saved
}

private var inQualifiedType = false
private def fromWithinQualifiedType[T](body: => T): T =
val saved = inQualifiedType
inQualifiedType = true
try body
finally inQualifiedType = saved

private var staged = StageKind.None
def withinStaged[T](kind: StageKind)(op: => T): T = {
val saved = staged
Expand Down Expand Up @@ -1879,12 +1886,22 @@ object Parsers {
t
}

/** WithType ::= AnnotType {`with' AnnotType} (deprecated)
*/
/** With qualifiedTypes enabled:
* WithType ::= AnnotType [`with' PostfixExpr]
*
* Otherwise:
* WithType ::= AnnotType {`with' AnnotType} (deprecated)
*/
def withType(): Tree = withTypeRest(annotType())

def withTypeRest(t: Tree): Tree =
if in.token == WITH then
if in.featureEnabled(Feature.qualifiedTypes) && in.token == WITH then
if inQualifiedType then t
else
in.nextToken()
val qualifier = postfixExpr()
QualifiedTypeTree(t, None, qualifier).withSpan(Span(t.span.start, qualifier.span.end))
else if in.token == WITH then
val withOffset = in.offset
in.nextToken()
if in.token == LBRACE || in.token == INDENT then
Expand Down Expand Up @@ -2032,6 +2049,7 @@ object Parsers {
* | ‘(’ ArgTypes ‘)’
* | ‘(’ NamesAndTypes ‘)’
* | Refinement
* | QualifiedType -- under qualifiedTypes
* | TypeSplice -- deprecated syntax (since 3.0.0)
* | SimpleType1 TypeArgs
* | SimpleType1 `#' id
Expand All @@ -2042,7 +2060,10 @@ object Parsers {
makeTupleOrParens(inParensWithCommas(argTypes(namedOK = false, wildOK = true, tupleOK = true)))
}
else if in.token == LBRACE then
atSpan(in.offset) { RefinedTypeTree(EmptyTree, refinement(indentOK = false)) }
if in.featureEnabled(Feature.qualifiedTypes) && in.lookahead.token == IDENTIFIER then
qualifiedType()
else
atSpan(in.offset) { RefinedTypeTree(EmptyTree, refinement(indentOK = false)) }
else if (isSplice)
splice(isType = true)
else
Expand Down Expand Up @@ -2205,6 +2226,19 @@ object Parsers {
else
inBraces(refineStatSeq())

/** QualifiedType ::= `{` Ident `:` Type `with` Block `}`
*/
def qualifiedType(): Tree =
val startOffset = in.offset
accept(LBRACE)
val id = ident()
accept(COLONfollow)
val tp = fromWithinQualifiedType(typ())
accept(WITH)
val qualifier = block(simplify = true)
accept(RBRACE)
QualifiedTypeTree(tp, Some(id), qualifier).withSpan(Span(startOffset, qualifier.span.end))

/** TypeBounds ::= [`>:' Type] [`<:' Type]
* | `^` -- under captureChecking
*/
Expand Down
4 changes: 4 additions & 0 deletions compiler/src/dotty/tools/dotc/printing/RefinedPrinter.scala
Original file line number Diff line number Diff line change
Expand Up @@ -814,6 +814,10 @@ class RefinedPrinter(_ctx: Context) extends PlainPrinter(_ctx) {
prefix ~~ idx.toString ~~ "|" ~~ tpeText ~~ "|" ~~ argsText ~~ "|" ~~ contentText ~~ postfix
case CapturesAndResult(refs, parent) =>
changePrec(GlobalPrec)("^{" ~ Text(refs.map(toText), ", ") ~ "}" ~ toText(parent))
case QualifiedTypeTree(parent, paramName, predicate) =>
paramName match
case Some(name) => "{" ~ toText(name) ~ ": " ~ toText(parent) ~ " with " ~ toText(predicate) ~ "}"
case None => toText(parent) ~ " with " ~ toText(predicate)
case ContextBoundTypeTree(tycon, pname, ownName) =>
toText(pname) ~ " : " ~ toText(tycon) ~ (" as " ~ toText(ownName) `provided` !ownName.isEmpty)
case _ =>
Expand Down
2 changes: 1 addition & 1 deletion compiler/src/dotty/tools/dotc/typer/ImportInfo.scala
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,7 @@ class ImportInfo(symf: Context ?=> Symbol,

/** Does this import clause or a preceding import clause enable `feature`?
*
* @param feature a possibly quailified name, e.g.
* @param feature a possibly qualified name, e.g.
* strictEquality
* experimental.genericNumberLiterals
*
Expand Down
4 changes: 4 additions & 0 deletions library/src/scala/annotation/qualified.scala
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
package scala.annotation

/** Annotation for qualified types. */
@experimental class qualified[T](predicate: T => Boolean) extends StaticAnnotation
4 changes: 4 additions & 0 deletions library/src/scala/runtime/stdLibPatches/language.scala
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,10 @@ object language:
@compileTimeOnly("`captureChecking` can only be used at compile time in import statements")
object captureChecking

/** Experimental support for qualified types */
@compileTimeOnly("`qualifiedTypes` can only be used at compile time in import statements")
object qualifiedTypes

/** Experimental support for automatic conversions of arguments, without requiring
* a language import `import scala.language.implicitConversions`.
*
Expand Down
4 changes: 4 additions & 0 deletions project/MiMaFilters.scala
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@ object MiMaFilters {
ProblemFilters.exclude[MissingClassProblem]("scala.runtime.stdLibPatches.language$experimental$betterFors$"),
ProblemFilters.exclude[MissingFieldProblem]("scala.runtime.stdLibPatches.language#experimental.quotedPatternsWithPolymorphicFunctions"),
ProblemFilters.exclude[MissingClassProblem]("scala.runtime.stdLibPatches.language$experimental$quotedPatternsWithPolymorphicFunctions$"),
ProblemFilters.exclude[MissingFieldProblem]("scala.runtime.stdLibPatches.language#experimental.qualifiedTypes"),
ProblemFilters.exclude[MissingClassProblem]("scala.runtime.stdLibPatches.language$experimental$qualifiedTypes$"),
ProblemFilters.exclude[DirectMissingMethodProblem]("scala.quoted.runtime.Patterns.higherOrderHoleWithTypes"),
),

Expand Down Expand Up @@ -72,6 +74,8 @@ object MiMaFilters {
ProblemFilters.exclude[MissingClassProblem]("scala.runtime.stdLibPatches.language$experimental$betterMatchTypeExtractors$"),
ProblemFilters.exclude[MissingClassProblem]("scala.runtime.stdLibPatches.language$experimental$modularity$"),
ProblemFilters.exclude[MissingClassProblem]("scala.runtime.stdLibPatches.language$experimental$namedTuples$"),
ProblemFilters.exclude[MissingFieldProblem]("scala.runtime.stdLibPatches.language#experimental.qualifiedTypes"),
ProblemFilters.exclude[MissingClassProblem]("scala.runtime.stdLibPatches.language$experimental$qualifiedTypes$"),
ProblemFilters.exclude[MissingFieldProblem]("scala.runtime.stdLibPatches.language.3.7-migration"),
ProblemFilters.exclude[MissingFieldProblem]("scala.runtime.stdLibPatches.language.3.7"),
ProblemFilters.exclude[MissingClassProblem]("scala.runtime.stdLibPatches.language$3$u002E7$"),
Expand Down
Loading

0 comments on commit d5021f8

Please sign in to comment.