Handle Future result from Future.sync computation

sdk/lib/async/future.dart

Index: sdk/lib/async/future.dart
diff --git a/sdk/lib/async/future.dart b/sdk/lib/async/future.dart
index 6b6c760dadb760119dcc4140a9ee492121fd5368..72918158b908691b61db201e27cd54cbd1742a90 100644
--- a/sdk/lib/async/future.dart
+++ b/sdk/lib/async/future.dart
@@ -128,7 +128,7 @@ abstract class Future<T> {
   factory Future.sync(computation()) {
     try {
       var result = computation();
-      return new Future<T>.value(result);
+      return new Future.value(result);

[kevmoo-old, 2013/09/17 17:21:41]

After I did the update, I decided I don't like this either.

[floitsch, 2013/09/17 17:59:44]

This Future<T> is correct. But the Future.value should not expect a T.
It means that the value will not be type-checked immediately but it will be
checked eventually (and we can help it by assigning it to T after the Future
check).

     } catch (error, stackTrace) {
       return new Future<T>.error(error, stackTrace);
     }
@@ -365,7 +365,7 @@ abstract class Future<T> {
  *
  * If you already have a Future, you probably don't need a Completer.
  * Instead, you can usually use [Future.then], which returns a Future:
- * 
+ *
  *     Future doStuff(){
  *       return someAsyncOperation().then((result) {
  *         // Do something.