Source in most stuff.

dart/frog/leg/scanner/scanner_task.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 ScannerTask extends CompilerTask {
   ScannerTask(Compiler compiler) : super(compiler);
   String get name() => 'Scanner';
 
   void scan(CompilationUnitElement compilationUnit) {
     measure(() {
       scanElements(compilationUnit);
       if (compilationUnit.kind === ElementKind.LIBRARY) {
         processScriptTags(compilationUnit);
       }
     });
   }
 
   void processScriptTags(LibraryElement library) {
+    LinkBuilder<ScriptTag> imports = new LinkBuilder<ScriptTag>();
+    Uri cwd = new Uri(scheme: 'file', path: compiler.currentDirectory);

[kasperl, 2012/02/15 09:22:30]

The named arguments are cool. Would it make sense to split this method into two
parts? One that returns the sources and imports and checks that the tags are
valid -- and one that does the readScript, importLibrary calls? You almost have
that split now.

[ahe, 2012/02/15 09:46:55]

I need to process source directives before I start importing. So it would look
something like this:

Link<ScriptTag> tags = checkTags();
for (ScriptTag tag in tags) {
  if (tag.isSource()) sourceScript(tag);
}
for (ScriptTag tag in tags) {
  if (tag.isImport()) importLibrary(tag);
}

Perhaps I can make this a little cleaner in a future CL.

+    Uri base = cwd.resolve(library.script.name.toString());
     for (ScriptTag tag in library.tags.reverse()) {
       SourceString argument = tag.argument.value.copyWithoutQuotes(1, 1);
-      Uri cwd = new Uri(scheme: 'file', path: compiler.currentDirectory);
-      Uri base = cwd.resolve(library.script.name.toString());
       Uri resolved = base.resolve(argument.toString());
       if (tag.isImport()) {
-        importLibrary(library, loadLibrary(resolved, tag), tag.prefix);
+        imports.addLast(tag);
       } else if (tag.isLibrary()) {
         if (library.libraryTag !== null) {
           compiler.cancel("duplicated library declaration", node: tag);
         } else {
           library.libraryTag = tag;
         }
       } else if (tag.isSource()) {
         Script script = compiler.readScript(resolved, tag);
         CompilationUnitElement unit =
           new CompilationUnitElement(script, library);
         compiler.withCurrentElement(unit, () => scan(unit));
       } else {
         compiler.cancel("illegal script tag: ${tag.tag}", node: tag);
       }
     }
+    for (ScriptTag tag in imports.toLink()) {

[ngeoffray, 2012/02/15 09:53:03]

Maybe add a comment on why you need to import last.

[ahe, 2012/02/15 15:08:42]

Done.

+      SourceString argument = tag.argument.value.copyWithoutQuotes(1, 1);
+      Uri resolved = base.resolve(argument.toString());
+      importLibrary(library, loadLibrary(resolved, tag), tag.prefix);
+    }
     if (compiler.coreLibrary !== null) {
       importLibrary(library, compiler.coreLibrary, null);
     }
   }
 
   void scanElements(CompilationUnitElement compilationUnit) {
     compiler.log("scanning $compilationUnit");
     Script script = compilationUnit.script;
     Token tokens;
     try {
@@ skipping 35 matching lines @@
     } else {
       for (Link<Element> link = imported.topLevelElements; !link.isEmpty();
            link = link.tail) {
         compiler.withCurrentElement(link.head, () {
             library.define(link.head, compiler);
           });
       }
     }
   }
 }