DelegatingReporter code review suggestions

Author: MasseGuillaume

bridge/src/main/scala/xsbt/CompilerInterface.scala

@@ -54,8 +54,10 @@ class CachedCompilerImpl(args: Array[String], output: Output, resident: Boolean)
   }
   private[this] def run(sources: List[File], changes: DependencyChanges, callback: AnalysisCallback, log: Logger, delegate: Reporter, compileProgress: CompileProgress): Unit = {
     debug(log, args.mkString("Calling Dotty compiler with arguments  (CompilerInterface):\n\t", "\n\t", ""))
-    val freshContext = (new ContextBase).initialCtx.fresh
-    val ctx = freshContext.setSbtCallback(callback).setReporter(DelegatingReporter(delegate))
+    val ctx = (new ContextBase).initialCtx.fresh
+      .setSbtCallback(callback)
+      .setReporter(new DelegatingReporter(delegate))
+
     val cl = getClass.getClassLoader.asInstanceOf[URLClassLoader]
 
     val reporter = DottyMain.process(commandArguments(sources.toArray), ctx)

bridge/src/main/scala/xsbt/DelegatingReporter.scala

@@ -8,11 +8,7 @@ import dotc._
 import reporting._
 import core.Contexts._
 
-import xsbti.Maybe
-
-object DelegatingReporter {
-  def apply(delegate: xsbti.Reporter) = new DelegatingReporter(delegate)
-}
+import xsbti.{Maybe, Position}
 
 final class DelegatingReporter(delegate: xsbti.Reporter) extends Reporter
   with UniqueMessagePositions
@@ -27,28 +23,24 @@ final class DelegatingReporter(delegate: xsbti.Reporter) extends Reporter
         case _: Reporter.Warning => xsbti.Severity.Warn
         case _                   => xsbti.Severity.Info
       }
-    val pos  = 
-      if(d.pos.exists) Some(d.pos)
+    val pos = 
+      if (d.pos.exists) Some(d.pos)
       else None
 
     val file = 
-      if(d.pos.source.file.exists) {
-        val r = d.pos.source.file.file 
-        if(r == null) None
-        else Some(r)
-      }
+      if (d.pos.source.file.exists) Option(d.pos.source.file.file)
       else None
 
     val offset0 = pos.map(_.point)
 
-    val position = new xsbti.Position {
+    val position = new Position {
       def line: Maybe[Integer] = maybe(pos.map(_.line))
-      def lineContent(): String = pos.map(_.lineContent).getOrElse("")
-      def offset(): xsbti.Maybe[Integer] = maybeInt(offset0)
-      def pointer(): xsbti.Maybe[Integer] = offset()
-      def pointerSpace(): xsbti.Maybe[String] = maybe(offset0.map(" " * _))
-      def sourceFile(): xsbti.Maybe[java.io.File] = maybe(file)
-      def sourcePath(): xsbti.Maybe[String] = maybe(file.map(_.getPath))
+      def lineContent: String = pos.map(_.lineContent).getOrElse("")
+      def offset: Maybe[Integer] = maybeInt(offset0)
+      def pointer: Maybe[Integer] = offset
+      def pointerSpace: Maybe[String] = maybe(offset0.map(" " * _))
+      def sourceFile: Maybe[java.io.File] = maybe(file)
+      def sourcePath: Maybe[String] = maybe(file.map(_.getPath))
     }
 
     delegate.log(position, d.message, severity)