pkg/path: Add type annotations to top-level and static fields.

lib/path.dart

Index: lib/path.dart
diff --git a/lib/path.dart b/lib/path.dart
index cfbaf11fe078336c0e95b93db7fe0b33a870132c..af9efe54e849ded0200f1f45b469c32ac3c35bed 100644
--- a/lib/path.dart
+++ b/lib/path.dart
@@ -54,13 +54,13 @@ export 'src/path_exception.dart';
 export 'src/style.dart';
 
 /// A default context for manipulating POSIX paths.
-final posix = new Context(style: Style.posix);
+final Context posix = new Context(style: Style.posix);
 
 /// A default context for manipulating Windows paths.
-final windows = new Context(style: Style.windows);
+final Context windows = new Context(style: Style.windows);
 
 /// A default context for manipulating URLs.
-final url = new Context(style: Style.url);
+final Context url = new Context(style: Style.url);

[nweiz, 2015/04/09 18:41:23]

I don't like this change. It adds verbosity without increasing readability. It's
already immediately clear to anyone looking at the code that these are Context
objects, since the word "Context" is right there.

[kevmoo, 2015/04/09 19:07:55]

It's not about code reading, it's about how tools (IDEs, docgen) surface the
code.

Right now the type is 'dynamic' which is valid for many things where the user
shouldn't rely on the return type.

We should be explicit that the return type IS Context.

Even in C#, which has support for compile-type inference requires public members
to be typed.

 
 /// The system path context.
 ///