Handle (missing) trailing slash in dartdoc

utils/apidoc/apidoc.dart

Index: utils/apidoc/apidoc.dart
diff --git a/utils/apidoc/apidoc.dart b/utils/apidoc/apidoc.dart
index bbd2f58a6fdd949c9b99cd644f21dfd49c9c03ee..fe12e4ee7185b48b99a2b10a83c3df11c275cbbb 100644
--- a/utils/apidoc/apidoc.dart
+++ b/utils/apidoc/apidoc.dart
@@ -64,7 +64,7 @@ void main() {
     }
   }
 
-  final libPath = doc.scriptDir.append('../../sdk/lib');
+  final libPath = doc.scriptDir.append('../../sdk/');
   final pkgPath = doc.scriptDir.append('../../pkg/');
 
   doc.cleanOutputDirectory(outputDir);
@@ -553,7 +553,7 @@ class Apidoc extends doc.Dartdoc {
   String _linkMember(MemberMirror member) {
     final typeName = member.owner.simpleName;
     var memberName = '$typeName.${member.simpleName}';
-    if (member is MethodMirror && (member.isConstructor || member.isFactory)) {

[Johnni Winther, 2012/11/05 13:42:50]

Dead code (and a resolution error - it's now called isFactoryConstructor)

[ahe, 2012/11/05 15:58:42]

What is dead code, and what is a resolution error? Did you mean warning?

[Johnni Winther, 2012/11/05 16:23:29]

The isFactory method has been renamed to isFactoryConstructor in a previous CL,
and yes, we're not in Java so that is just a dynamic NoSuchMethodError. 

In any case, a factory is also a constructor, so the code is dead.

[ahe, 2012/11/05 16:46:08]

Terminology by example:

if (false) {
  print('This is dead code');
}

if (x < 2 || x < 1) {
  print("This is not dead code, it's just a redundant test");
}

So this expression "member is MethodMirror && (member.isConstructor ||
member.isFactory)" does not contain dead code, and it throws a dynamic error if
[member] is a MethodMirror that isn't a constructor.

+    if (member is MethodMirror && member.isConstructor) {
       final separator = member.constructorName == '' ? '' : '.';
       memberName = 'new $typeName$separator${member.constructorName}';
     }