Polishings

Author: odersky

compiler/src/dotty/tools/dotc/core/Decorators.scala

@@ -11,7 +11,7 @@ import printing.{ Printer, Showable }, printing.Formatting._, printing.Texts._
 import transform.MegaPhase
 import reporting.{Message, NoExplanation}
 
-/** This object provides useful implicit decorators for types defined elsewhere */
+/** This object provides useful extension methods for types defined elsewhere */
 object Decorators {
 
   /** Extension methods for toType/TermName methods on PreNames.

compiler/src/dotty/tools/dotc/reporting/Message.scala

@@ -15,12 +15,12 @@ import scala.annotation.threadUnsafe
 
 /** ## Tips for error message generation
  *
- *  - You can use the `em` interpolator for error messages. It's defined in decorators.
+ *  - You can use the `em` interpolator for error messages. It's defined in core.Decorators.
  *  - You can also use a simple string argument for `error` or `warning` (not for the other variants),
  *    but the string should not be interpolated or composed of objects that require a
  *    Context for evaluation.
  *  - When embedding interpolated substrings defined elsewhere in error messages,
- *    make sure they are defined as def's instead of vals. That way, the
+ *    use `i` and make sure they are defined as def's instead of vals. That way, the
  *    possibly expensive interpolation will performed only in the case where the message
  *    is eventually printed. Note: At least during typer, it's common for messages
  *    to be discarded without being printed. Also, by making them defs, you ensure that

compiler/src/dotty/tools/package.scala

@@ -1,10 +1,6 @@
 package dotty
 
 package object tools {
-  // Ensure this object is already classloaded, since it's only actually used
-  // when handling stack overflows and every operation (including class loading)
-  // risks failing.
-  dotty.tools.dotc.core.handleRecursive
 
   val ListOfNil: List[Nil.type] = Nil :: Nil
 
@@ -49,4 +45,9 @@ package object tools {
       val e = if msg == null then AssertionError() else AssertionError("assertion failed: " + msg)
       e.setStackTrace(Array())
       throw e
-}
+
+  // Ensure this object is already classloaded, since it's only actually used
+  // when handling stack overflows and every operation (including class loading)
+  // risks failing.
+  dotty.tools.dotc.core.handleRecursive
+ }