Spelling

Author: Jentsch

compiler/src/dotty/tools/dotc/typer/Applications.scala

@@ -1171,7 +1171,7 @@ trait Applications extends Compatibility {
    *  type by means of a type ascription, as long as the widened type is
    *  still compatible with the expected type.
    *  The underlying type is the intersection of all class parents of the
-   *  orginal type.
+   *  original type.
    */
   def widenEnumCase(tree: Tree, pt: Type)(using Context): Tree =
     val sym = tree.symbol
@@ -1383,7 +1383,7 @@ trait Applications extends Compatibility {
 
         // named pattern
         // TODO: Errors report the wrong position
-        // TODO: Use propper error reporting
+        // TODO: Use proper error reporting
         // TODO: Maybe the 'reorder' method above can be reused, or be template
         if (bunchedArgs != Nil && argTypes != Nil) {
 
@@ -1401,28 +1401,29 @@ trait Applications extends Compatibility {
                   .map(_.group(1))
                   .zipWithIndex
                   .toMap
-              // TODO: The test should accutally be: is unapplyFn generated by the compiler because we have a case class?
+              // TODO: The test should actually be: is unapplyFn generated by the compiler because we have a case class?
               else if unapplyFn.tpe.widen.paramInfoss.head.head.typeSymbol.is(CaseClass) then
                 unapplyFn.tpe.widen.paramInfoss.head.head.fields.map(_.name.toString).zipWithIndex.toMap
               else
                 // TODO: Report member
                 report.error("Doesn't support named patterns")
                 Map.empty
+            // TODO: Is is necessary to convert names to strings?
             .compose{ case name: Name => name.show }
 
           val namedArgs = bunchedArgs
             .flatMap {
               case pattern @ NamedArg(positionOfName(_), _) => Seq(pattern)
-              case pattern @ NamedArg(unkownName, _) =>
-                report.error(s"'${unkownName.show}' is unknown", pattern)
+              case pattern @ NamedArg(unknownName, _) =>
+                report.error(s"'${unknownName.show}' is unknown", pattern)
                 Seq.empty
               case unnamedArgument =>
                 report.error("Only named arguments allowed here", unnamedArgument)
                 Seq.empty
             }
             .groupMapReduce(arg => positionOfName(arg.name))(identity)(
               (first, second) => {
-                // TODO: Document design decission here
+                // TODO: Document design decision here
                 report.error(s"'${second.name}' was already used before", second);
                 first
               })