Find diagnostic locations for use in new compiler API.

dart/frog/leg/compiler.dart

 // Copyright (c) 2012, 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.
 
 class WorkItem {
   final Element element;
   TreeElements resolutionTree;
   Function run;
   Map<int, BailoutInfo> bailouts = null;
   bool allowSpeculativeOptimization = true;
@@ skipping 107 matching lines @@
   }
 
   void internalError(String message,
                      [Node node, Token token, HInstruction instruction,
                       Element element]) {
     cancel("Internal Error: $message", node, token, instruction, element);
   }
 
   void cancel([String reason, Node node, Token token,
                HInstruction instruction, Element element]) {
+    SourceSpan span = const SourceSpan(null, null, null);
+    if (node !== null) {
+      span = spanFromNode(node);
+    } else if (token !== null) {
+      span = spanFromTokens(token, token);
+    } else if (instruction !== null) {
+      span = spanFromElement(currentElement);
+    } else if (element !== null) {
+      span = spanFromElement(element);
+    }
+    reportDiagnostic(span, reason, true);
     throw new CompilerCancelledException(reason);
   }
 
   void log(message) {
-    // Do nothing.
+    reportDiagnostic(null, message, false);
   }
 
   void enqueue(WorkItem work) {
     worklist.add(work);
   }
 
   bool run(Uri uri) {
     try {
       runCompiler(uri);
     } catch (CompilerCancelledException exception) {
@@ skipping 216 matching lines @@
                               () => resolver.resolveSignature(element));
   }
 
   Object compileVariable(VariableElement element) {
     return withCurrentElement(element, () {
         compile(new WorkItem.toCompile(element));
         return constantHandler.compileVariable(element);
       });
   }
 
-  reportWarning(Node node, var message) {}
+  reportWarning(Node node, var message) {
+    SourceSpan span = spanFromNode(node);
+    reportDiagnostic(span, message.toString(), false);
+  }
 
-  reportError(Node node, var message) => cancel(message.toString(), node: node);
+  reportError(Node node, var message) {
+    SourceSpan span = spanFromNode(node);
+    reportDiagnostic(span, message.toString(), true);
+    throw new CompilerCancelledException(message.toString());
+  }
+
+  abstract void reportDiagnostic(SourceSpan span, String message, bool fatal);
+
+  SourceSpan spanFromTokens(Token begin, Token end) {
+    if (begin === null || end === null) {
+      throw 'cannot find tokens to produce error message';

[ngeoffray, 2012/03/10 10:36:58]

Should that be return const SourceSpan(null, null, null) instead?

[ahe, 2012/03/10 10:50:58]

I'm really torn about this. It really is an internal error in the compiler but
we can do better in many cases. For example: we almost always know the current
URI. Often, we may be able to get the begin token, but not the end token. I'll
add a TODO and mull it over.

+    }
+    final startOffset = begin.charOffset;
+    // TODO(ahe): Compute proper end offset.
+    final endOffset =
+      (end.next !== null) ? end.next.charOffset - 1 : startOffset + 1;
+    Uri uri = currentElement.getCompilationUnit().script.uri;
+    return new SourceSpan(uri, startOffset, endOffset);
+  }
+
+  SourceSpan spanFromNode(Node node) {
+    return spanFromTokens(node.getBeginToken(), node.getEndToken());
+  }
+
+  SourceSpan spanFromElement(Element element) {
+    if (element.position() === null) {
+      // Sometimes, the backend fakes up elements that have no
+      // position. So we use the enclosing element instead. It is
+      // not a good error location, but cancel really is "internal
+      // error" or "not implemented yet", so the vicinity is good
+      // enough for now.
+      element = element.enclosingElement;
+      // TODO(ahe): I plan to overhaul this infrastructure anyways.
+    }
+    if (element === null) {
+      element = currentElement;
+    }
+    Token position = element.position();
+    if (position === null) {
+      // TODO(ahe): Find the enclosing library.
+      return const SourceSpan(null, null, null);
+    }
+    return spanFromTokens(position, position);
+  }
 
   Script readScript(Uri uri, [ScriptTag node]) {
     unimplemented('Compiler.readScript');
   }
 
   String get legDirectory() {
     unimplemented('Compiler.legDirectory');
   }
 
   Element findHelper(SourceString name) => jsHelperLibrary.find(name);
@@ skipping 41 matching lines @@
 
 // TODO(ahe): Remove when the VM supports implicit interfaces.
 class LTracer implements Tracer {
   const LTracer();
   final bool enabled = false;
   void traceGraph(String name, var graph) {
   }
   void close() {
   }
 }
+
+class SourceSpan {
+  final Uri uri;
+  final int begin;
+  final int end;
+
+  const SourceSpan(this.uri, this.begin, this.end);
+}