Dartdoc and Apidoc updated to use dart2js through the mirror system.

lib/dartdoc/utils.dart

Index: lib/dartdoc/utils.dart
diff --git a/lib/dartdoc/utils.dart b/lib/dartdoc/utils.dart
index b2e5758375bb9ca3feb3b83d7bbd78b94813220b..45df37854c77b1d641e119c5ee325052b59d6004 100644
--- a/lib/dartdoc/utils.dart
+++ b/lib/dartdoc/utils.dart
@@ -47,20 +47,20 @@ String unindent(String text, int indentation) {
 }
 
 /** Sorts the map by the key, doing a case-insensitive comparison. */
-List orderByName(Map<String, Dynamic> map) {
-  List keys = map.getKeys();
-  keys.sort((a, b) {
+List orderByName(List list) {

[Lasse Reichstein Nielsen, 2012/07/09 10:39:33]

Put a generic type on the List types here. It's part of the interface contract. 
It also means I'll actually have a clue what the type of its elements are,
because it isn't mentioned ANYWHERE in this method.

And add types everywhere else too, please!!! It helps reading for me. 
Only ever use "var" when the RHS is a constructor call. Otherwise it is not
clear what type is expected, and I can't read the following code.

[Johnni Winther, 2012/07/09 14:57:18]

Done.

+  final elements = [];
+  for (var element in list) {
+    elements.add(element);
+  }

[Lasse Reichstein Nielsen, 2012/07/09 10:39:33]

final elements = new List<whateveritis>.from(list);

[Johnni Winther, 2012/07/09 14:57:18]

Done.

+  elements.sort((a, b) {
     // Hack: make sure $dom methods show up last in the list not first.
     String transformName(String name) =>
       name.toUpperCase().replaceFirst(const RegExp('\\\$DOM_'), 'zzzz');

[Lasse Reichstein Nielsen, 2012/07/09 10:39:33]

Use raw strings for regexp literals: const RegExp(@'\$DOM').

This seems uncredibly slow - you create two new strings *per comparison*. Try:

(a,b) {
  bool doma = a.indexOf(@"$DOM") >= 0;
  bool domb = a.indexOf(@"$DOM") >= 0;
  return doma == domb ? a.compareTo(b) : doma ? 1 : -1;
}

(Or, if the "$DOM" is always at a specific position, you might be able to use
.startsWith instead of .indexOf, or use lastIndexOf it's usually closer to the
end than to the start.

[Johnni Winther, 2012/07/09 14:57:18]

Done.

 
-    return transformName(a).compareTo(transformName(b));
+    return transformName(a.simpleName()).compareTo(

[kasperl, 2012/07/06 12:40:41]

Maybe do the transform before calling compareTo and use two extra local
variables?

[Johnni Winther, 2012/07/09 14:57:18]

Done.

+           transformName(b.simpleName()));
   });
-  final values = [];
-  for (var k in keys) {
-    values.add(map[k]);
-  }
-  return values;
+  return elements;
 }
 
 /**
@@ -70,6 +70,12 @@ List orderByName(Map<String, Dynamic> map) {
 String joinWithCommas(List<String> items, [String conjunction = 'and']) {
   if (items.length == 1) return items[0];
   if (items.length == 2) return "${items[0]} $conjunction ${items[1]}";
-  return Strings.join(items.getRange(0, items.length - 1), ', ') +
-    ', $conjunction ' + items[items.length - 1];
+  return '${Strings.join(items.getRange(0, items.length - 1), ', ')}'
+    ', $conjunction ${items[items.length - 1]}';
+}
+
+void writeString(File file, String text) {
+  var randomAccessFile = file.openSync(FileMode.WRITE);
+  randomAccessFile.writeStringSync(text);
+  randomAccessFile.closeSync();
 }