Change Driver API to not need definition of newCompiler

Author: felixmulder

compiler/src/dotty/tools/dotc/Bench.scala

@@ -16,8 +16,6 @@ object Bench extends Driver {
 
   @sharable private var numRuns = 1
 
-  def newCompiler(implicit ctx: Context): Compiler = new Compiler
-
   private def ntimes(n: Int)(op: => Reporter): Reporter =
     (emptyReporter /: (0 until n)) ((_, _) => op)
 

compiler/src/dotty/tools/dotc/Driver.scala

@@ -13,9 +13,9 @@ import scala.util.control.NonFatal
  *  process, but in most cases you only need to call [[process]] on the
  *  existing object [[Main]].
  */
-abstract class Driver extends DotClass {
+class Driver extends DotClass {
 
-  protected def newCompiler(implicit ctx: Context): Compiler
+  protected def newCompiler(implicit ctx: Context): Compiler = new Compiler
 
   protected def emptyReporter: Reporter = new StoreReporter(null)
 

compiler/src/dotty/tools/dotc/Main.scala

@@ -4,6 +4,4 @@ package dotc
 import core.Contexts.Context
 
 /** Main class of the `dotc` batch compiler. */
-object Main extends Driver {
-  override def newCompiler(implicit ctx: Context): Compiler = new Compiler
-}
+object Main extends Driver

compiler/src/dotty/tools/dotc/Resident.scala

@@ -27,8 +27,6 @@ class Resident extends Driver {
 
   object residentCompiler extends Compiler
 
-  override def newCompiler(implicit ctx: Context): Compiler = ???
-
   override def sourcesRequired = false
 
   private val quit = ":q"

compiler/src/dotty/tools/dotc/interactive/InteractiveDriver.scala

@@ -28,8 +28,6 @@ class InteractiveDriver(settings: List[String]) extends Driver {
   import tpd._
   import InteractiveDriver._
 
-  // FIXME: Change the Driver API to not require implementing this method
-  override protected def newCompiler(implicit ctx: Context): Compiler = ???
   override def sourcesRequired = false
 
   private val myInitCtx: Context = {

compiler/test/dotty/tools/vulpix/ParallelTesting.scala

@@ -373,10 +373,8 @@ trait ParallelTesting extends RunnerOrchestration { self =>
           if (suppressErrors || suppressAllOutput) ERROR + 1 else ERROR)
 
       val driver =
-        if (times == 1) new Driver { def newCompiler(implicit ctx: Context) = new Compiler }
+        if (times == 1) new Driver
         else new Driver {
-          def newCompiler(implicit ctx: Context) = new Compiler
-
           private def ntimes(n: Int)(op: Int => Reporter): Reporter =
             (emptyReporter /: (1 to n)) ((_, i) => op(i))