- Add getters for the current packageRoot or packageMap.

pkg/docgen/lib/src/generator.dart

 // Copyright (c) 2014, 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.
 
 library docgen.generator;
 
 import 'dart:async';
 import 'dart:collection';
 import 'dart:convert';
 import 'dart:io';
@@ skipping 248 matching lines @@
 /// available on the file system.
 void _ensureOutputDirectory(String outputDirectory) {
   _outputDirectory = outputDirectory;
   var dir = new Directory(_outputDirectory);
   if (dir.existsSync()) dir.deleteSync(recursive: true);
 }
 
 /// Analyzes set of libraries and provides a mirror system which can be used
 /// for static inspection of the source code.
 Future<MirrorSystem> analyzeLibraries(List<Uri> libraries, String
-    libraryRoot, {String packageRoot}) {
+    libraryRoot, {String packageRoot}) async {
   SourceFileProvider provider = new CompilerSourceFileProvider();
   api.DiagnosticHandler diagnosticHandler = new FormattingDiagnosticHandler(
       provider)
       ..showHints = false
       ..showWarnings = false;
   Uri libraryUri = new Uri.file(appendSlash(libraryRoot));
   Uri packageUri = null;
-  if (packageRoot == null) {
-    packageRoot = Platform.packageRoot;
+  if (packageRoot != null) {
+    packageUri = new Uri.file(appendSlash(packageRoot));
+  } else {
+    packageUri = await Platform.packageRoot;
   }
-  packageUri = new Uri.file(appendSlash(packageRoot));
-  return dart2js.analyze(libraries, libraryUri, packageUri,
+  try {
+    return await dart2js.analyze(libraries, libraryUri, packageUri,
       provider.readStringFromUri, diagnosticHandler, ['--preserve-comments',
-      '--categories=Client,Server'])..catchError((error) {

[Lasse Reichstein Nielsen, 2015/09/22 09:48:25]

I'm not sure what the ".." does here. It's ... suspicious. There is no reason to
not use just ".".

-        logger.severe('Error: Failed to create mirror system. ');
-        // TODO(janicejl): Use the stack trace package when bug is resolved.
-        // Currently, a string is thrown when it fails to create a mirror
-        // system, and it is not possible to use the stack trace. BUG(#11622)
-        // To avoid printing the stack trace.
-        exit(1);
-      });
+      '--categories=Client,Server']);
+  } catch (error) {
+    logger.severe('Error: Failed to create mirror system. ');
+    // TODO(janicejl): Use the stack trace package when bug is resolved.
+    // Currently, a string is thrown when it fails to create a mirror
+    // system, and it is not possible to use the stack trace. BUG(#11622)
+    // To avoid printing the stack trace.
+    exit(1);
+  }

[Lasse Reichstein Nielsen, 2015/09/22 09:48:25]

This is slightly different from the original because the original returns a
future without a catchError. That's not something that can be simulated using an
async function (it splits the future).
It's also completely unnecessary, so I think the change is fine.

 }
 
 /// For this run of docgen, determine the packageRoot value.
 ///
 /// If packageRoot is not explicitly passed, we examine the files we're
 /// documenting to attempt to find a package root.
 String _obtainPackageRoot(String packageRoot, bool parseSdk,
     List<String> files) {
   if (packageRoot == null && !parseSdk) {
     var type = FileSystemEntity.typeSync(files.first);
@@ skipping 177 matching lines @@
   * [Samples](http://www.dartlang.org/samples/)
   * [A Tour of the Dart Libraries](http://www.dartlang.org/docs/dart-up-and-running/contents/ch03.html)
 
 This API reference is automatically generated from the source code in the
 [Dart project](https://code.google.com/p/dart/).
 If you'd like to contribute to this documentation, see
 [Contributing](https://code.google.com/p/dart/wiki/Contributing)
 and
 [Writing API Documentation](https://code.google.com/p/dart/wiki/WritingApiDocumentation).
 """;