DartC User Warning Framework

compiler/java/com/google/dart/runner/DartRunner.java

Index: compiler/java/com/google/dart/runner/DartRunner.java
diff --git a/compiler/java/com/google/dart/runner/DartRunner.java b/compiler/java/com/google/dart/runner/DartRunner.java
index cabe6214126772afdd09707f1e2c720d71106dbd..87c12a1582712c505b08ddeaad5a76de02d0abf0 100644
--- a/compiler/java/com/google/dart/runner/DartRunner.java
+++ b/compiler/java/com/google/dart/runner/DartRunner.java
@@ -13,7 +13,6 @@ import com.google.dart.compiler.CommandLineOptions;
 import com.google.dart.compiler.CommandLineOptions.DartRunnerOptions;
 import com.google.dart.compiler.CompilerConfiguration;
 import com.google.dart.compiler.DartArtifactProvider;
-import com.google.dart.compiler.DartCompilationError;
 import com.google.dart.compiler.DartCompiler;
 import com.google.dart.compiler.DartCompilerListener;
 import com.google.dart.compiler.DefaultCompilerConfiguration;
@@ -75,7 +74,6 @@ public class DartRunner {
   public static void main(String[] args) {
     try {
       boolean runBatch = false;
-      final List<LibrarySource> imports = Collections.emptyList();
       DartRunnerOptions options = processCommandLineOptions(args);
       if (options.shouldBatch()) {
         runBatch = true;
@@ -130,16 +128,12 @@ public class DartRunner {
         {
           ((DefaultErrorFormatter) formatter).setOutputStream(stderr);

[zundel, 2011/10/27 16:19:54]

this is pretty ugly, could you make an overload constructor to set the output
stream instead?

         }
-        @Override
-        public void compilationWarning(DartCompilationError event) {
-          compilationError(event);
-        }
       };
 
     CompilationResult compiled;
     compiled = compileApp(app, imports, options, listener);
 
-    if (listener.getProblemCount() != 0) {
+    if (listener.getErrorCount() != 0) {
       throw new RunnerError("Compilation failed.");
     }