Merge pull request #506 from dotty-staging/update-comments

Update comments on ParamForwarding and SuperAccessors.

Author: odersky

src/dotty/tools/dotc/Run.scala

@@ -10,6 +10,7 @@ import reporting.Reporter
 import transform.TreeChecker
 import java.io.{BufferedWriter, OutputStreamWriter}
 import scala.reflect.io.VirtualFile
+import scala.util.control.NonFatal
 
 class Run(comp: Compiler)(implicit ctx: Context) {
 
@@ -27,9 +28,13 @@ class Run(comp: Compiler)(implicit ctx: Context) {
     }
   }
 
-  def compile(fileNames: List[String]): Unit = {
+  def compile(fileNames: List[String]): Unit = try {
     val sources = fileNames map getSource
     compileSources(sources)
+  } catch {
+    case NonFatal(ex) =>
+      println(s"exception occurred while compiling $units%, %")
+      throw ex
   }
 
   /** TODO: There's a fundamental design problem here: We assmble phases using `squash`

src/dotty/tools/dotc/transform/ParamForwarding.scala

@@ -11,7 +11,7 @@ import Contexts._, Types._, Symbols._, Flags._, TypeUtils._, DenotTransformers._
  *
  *  if
  *  (1) x is forwarded in the supercall to a parameter that's also named `x`
- *  (2) the superclass parameter accessor for `x` is accessible from the current class to
+ *  (2) the superclass parameter accessor for `x` is accessible from the current class
  *  change the accessor to
  *
  *      def x: T = super.x.asInstanceOf[T]

src/dotty/tools/dotc/transform/SuperAccessors.scala

@@ -48,7 +48,7 @@ class SuperAccessors(thisTransformer: DenotTransformer) {
      *  of adding accessors. For instance, super calls from these regions
      *  always have to go through an accessor.
      *
-     *  The `invalidOwner` field, if different from NoSymbol,
+     *  The `invalidEnclClass` field, if different from NoSymbol,
      *  contains the symbol that is not a valid owner.
      */
     private var invalidEnclClass: Symbol = NoSymbol