fixes #219, able to compile multiple entry points

lib/src/report.dart

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /// Summarizes the information produced by the checker.
 library dev_compiler.src.report;
 
 import 'dart:math' show max;
 
 import 'package:analyzer/src/generated/engine.dart' show AnalysisContext;
 import 'package:analyzer/src/generated/error.dart';
 import 'package:logging/logging.dart';
 import 'package:path/path.dart' as path;
 import 'package:source_span/source_span.dart';
 
 import 'utils.dart';
 import 'summary.dart';
 
 final _checkerLogger = new Logger('dev_compiler.checker');
 
-SourceSpanWithContext _toSpan(AnalysisContext context, AnalysisError error) {
-  var source = error.source;
-  var lineInfo = context.computeLineInfo(source);
-  var content = context.getContents(source).data;
-  var start = error.offset;
-  var end = start + error.length;
-  return createSpanHelper(lineInfo, start, end, source, content);
-}
 /// Simple reporter that logs checker messages as they are seen.
 class LogReporter implements AnalysisErrorListener {
   final AnalysisContext _context;
   final bool useColors;
 
   LogReporter(this._context, {this.useColors: false});
 
   void onError(AnalysisError error) {
     var level = _severityToLevel[error.errorCode.errorSeverity];
 
     // Upgrade analyzer warnings to errors.
     // TODO(jmesserly: reconcile this...
     if (!error.errorCode.name.startsWith('dev_compiler.') &&
         level == Level.WARNING) {
       level = Level.SEVERE;
     }
 
-    var color = useColors ? colorOf(level.name) : null;
+    // TODO(jmesserly): figure out what to do with the error's name.
+    var lineInfo = _context.computeLineInfo(error.source);
+    var location = lineInfo.getLocation(error.offset);
 
-    // TODO(jmesserly): figure out what to do with the error's name.
-    var text = '[${errorCodeName(error.errorCode)}] ' + error.message;
-    text = _toSpan(_context, error).message(text, color: color);
+    // [warning] 'foo' is not a... (/Users/.../tmp/foo.dart, line 1, col 2)
+    var text = new StringBuffer()
+      ..write('[${errorCodeName(error.errorCode)}] ')
+      ..write(error.message)
+      ..write(' (${path.prettyUri(error.source.uri)}')
+      ..write(', line ${location.lineNumber}, col ${location.columnNumber})');
 
     // TODO(jmesserly): just print these instead of sending through logger?
     _checkerLogger.log(level, text);
   }
 }
 
 // TODO(jmesserly): remove log levels, instead just use severity.
 const _severityToLevel = const {
   ErrorSeverity.ERROR: Level.SEVERE,
   ErrorSeverity.WARNING: Level.WARNING,
@@ skipping 33 matching lines @@
 
     var span = _toSpan(_context, error);
     var summary = _getIndividualSummary(error.source.uri);
     if (summary is LibrarySummary) {
       summary.countSourceLines(_context, error.source);
     }
     summary.messages.add(new MessageSummary(errorCodeName(code),
         code.errorSeverity.displayName, span, error.message));
   }
 
+  // TODO(jmesserly): fix to not depend on SourceSpan. This will be really slow
+  // because it will reload source text from disk, for every single message...
+  SourceSpanWithContext _toSpan(AnalysisContext context, AnalysisError error) {
+    var source = error.source;
+    var lineInfo = context.computeLineInfo(source);
+    var content = context.getContents(source).data;
+    var start = error.offset;
+    var end = start + error.length;
+    return createSpanHelper(lineInfo, start, end, source, content);
+  }
+
   void clearLibrary(Uri uri) {
     (_getIndividualSummary(uri) as LibrarySummary).clear();
   }
 
   void clearHtml(Uri uri) {
     HtmlSummary htmlSummary = result.loose['$uri'];
     if (htmlSummary != null) htmlSummary.messages.clear();
   }
 }
 
@@ skipping 186 matching lines @@
 
   visitMessage(MessageSummary message) {
     var kind = message.kind;
     errorCount.putIfAbsent(currentPackage, () => <String, int>{});
     errorCount[currentPackage].putIfAbsent(kind, () => 0);
     errorCount[currentPackage][kind]++;
     totals.putIfAbsent(kind, () => 0);
     totals[kind]++;
   }
 }