diff options
author | Paul Phillips <paulp@improving.org> | 2010-03-25 19:55:53 +0000 |
---|---|---|
committer | Paul Phillips <paulp@improving.org> | 2010-03-25 19:55:53 +0000 |
commit | 6ee24a3c5d31ef769e47410d8806a5f4bd7a06bf (patch) | |
tree | e0e5352fba603060439c6fe9e1d024192eb32bd5 /src/compiler/scala/tools/nsc/interactive | |
parent | c0b615fe80fd052d5c87f39cfa6626da43c03669 (diff) | |
download | scala-6ee24a3c5d31ef769e47410d8806a5f4bd7a06bf.tar.gz scala-6ee24a3c5d31ef769e47410d8806a5f4bd7a06bf.tar.bz2 scala-6ee24a3c5d31ef769e47410d8806a5f4bd7a06bf.zip |
While working on partest discovered that Compil...
While working on partest discovered that CompilerCommand ignores half
its constructor arguments and a couple dozen places blithely pass it
those arguments as if they're being used. Then there were setups like
this:
class OfflineCompilerCommand(
arguments: List[String],
settings: Settings,
error: String => Unit,
interactive: Boolean)
extends CompilerCommand(arguments, new Settings(error), error, false)
Hey offline compiler command, why throw away the perfectly good settings
you were given? Ever heard 'reduce, reuse, recycle'? How did you ever
work... or do you? No review.
Diffstat (limited to 'src/compiler/scala/tools/nsc/interactive')
-rw-r--r-- | src/compiler/scala/tools/nsc/interactive/BuildManager.scala | 4 | ||||
-rw-r--r-- | src/compiler/scala/tools/nsc/interactive/REPL.scala | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/src/compiler/scala/tools/nsc/interactive/BuildManager.scala b/src/compiler/scala/tools/nsc/interactive/BuildManager.scala index 46930d398f..302bba0e07 100644 --- a/src/compiler/scala/tools/nsc/interactive/BuildManager.scala +++ b/src/compiler/scala/tools/nsc/interactive/BuildManager.scala @@ -69,7 +69,7 @@ object BuildManagerTest extends EvalLoop { val settings = new Settings(error) settings.Ybuildmanagerdebug.value = true - val command = new CompilerCommand(args.toList, settings, error, false) + val command = new CompilerCommand(args.toList, settings) // settings.make.value = "off" // val buildManager: BuildManager = new SimpleBuildManager(settings) val buildManager: BuildManager = new RefinedBuildManager(settings) @@ -79,7 +79,7 @@ object BuildManagerTest extends EvalLoop { // enter resident mode loop { line => val args = line.split(' ').toList - val command = new CompilerCommand(args, settings, error, true) + val command = new CompilerCommand(args, settings) buildManager.update(command.files, Set.empty) } diff --git a/src/compiler/scala/tools/nsc/interactive/REPL.scala b/src/compiler/scala/tools/nsc/interactive/REPL.scala index f3b1900ef2..5589ddb9b1 100644 --- a/src/compiler/scala/tools/nsc/interactive/REPL.scala +++ b/src/compiler/scala/tools/nsc/interactive/REPL.scala @@ -28,7 +28,7 @@ object REPL { def process(args: Array[String]) { val settings = new Settings(error) reporter = new ConsoleReporter(settings) - val command = new CompilerCommand(args.toList, settings, error, false) + val command = new CompilerCommand(args.toList, settings) if (command.settings.version.value) reporter.info(null, versionMsg, true) else { |