Mirrors prototype added to dartdoc.

lib/dartdoc/mirrors/mirrors_util.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.
+
+#library('mirrors.util');
+
+#import('mirrors.dart');
+
+//------------------------------------------------------------------------------
+// Utility functions for using the Mirror API
+//------------------------------------------------------------------------------
+
+/**
+ * Returns an iterable over the type declarations directly inheriting from
+ * the declaration of this type.
+ */
+Iterable<InterfaceMirror> computeSubdeclarations(MirrorSystem system,
+                                                 InterfaceMirror type) {
+  type = type.declaration;
+  var subtypes = <InterfaceMirror>[];
+  system.libraries().forEach((_, library) {
+    for (InterfaceMirror otherType in library.types().getValues()) {
+      var superClass = otherType.superclass();
+      if (superClass !== null) {
+        superClass = superClass.declaration;
+        if (type.library() === superClass.library()) {
+          if (superClass == type) {
+             subtypes.add(otherType);
+          }
+        }
+      }
+      final superInterfaces = otherType.interfaces().getValues();
+      for (InterfaceMirror superInterface in superInterfaces) {
+        superInterface = superInterface.declaration;
+        if (type.library() === superInterface.library()) {
+          if (superInterface == type) {
+            subtypes.add(otherType);
+          }
+        }
+      }
+    }
+  });
+  return subtypes;
+}
+
+/**
+ * Finds the mirror in [map] by the simple name [name]. If [constructorName] or
+ * [operatorName] is provided, a constructor/operator method by that name is
+ * returned.

[Lasse Reichstein Nielsen, 2012/07/04 10:58:59]

If constructorName is given, but name matches a non-Method, that non-Method is
returned. I.e., constructorName is ignored, even though it seems like the user
had an intent with it. Is this intentional?

[Johnni Winther, 2012/07/04 13:19:17]

It wasn't. The test has been rewritten.

+ */
+Mirror findMirror(Map<Object,Mirror> map, String name,
+                  [String constructorName, String operatorName]) {
+  var mirror = null;
+  map.forEach((_,m) {

[Lasse Reichstein Nielsen, 2012/07/04 10:58:59]

space after comma, give "m" a better name and preferably a type, e.g., "(_,
Mirror mirror)" (except that "mirror" is already used).
 
We really need a find on maps. Something like:
  Map<K,V> {
    K findMatch(bool predicate(K key, V value));

Perhaps use the values collection instead:
 
 for (Mirror mirror in map.getValues()) {
   if (...) {
     return mirror;
   }
 }
 return null;

[Johnni Winther, 2012/07/04 13:19:17]

Done.

+    bool found = false;
+    if (m.simpleName() == name) {
+      found = true;

[Lasse Reichstein Nielsen, 2012/07/04 10:58:59]

Seems convoluted, but I guess that's actually for readability.
How about:
  if (m is! MethodMirror ||
      ((constructorName == null || 
        constructorName == m.constructorName) &&
       (operatorName == null ||
        operatorName == m.operatorName))) {
    mirror = m;
 } 

[Johnni Winther, 2012/07/04 13:19:17]

This wouldn't work for if [m] is a FieldMirror and [constructorName] is
non-null.

+      if (constructorName !== null && m is MethodMirror) {
+        if (constructorName != m.constructorName) {
+          found = false;
+        }
+      }
+      if (operatorName !== null && m is MethodMirror) {
+        if (operatorName != m.operatorName) {
+          found = false;
+        }
+      }
+    }
+    if (found) {
+      mirror = m;
+    }
+  });
+  return mirror;
+}