Chromium Code Reviews| Index: corelib/src/implementation/future_implementation.dart |
| =================================================================== |
| --- corelib/src/implementation/future_implementation.dart (revision 855) |
| +++ corelib/src/implementation/future_implementation.dart (working copy) |
| @@ -75,7 +75,7 @@ |
| } |
| bool get hasValue() { |
| - return isComplete && exception == null; |
| + return isComplete && exception === null; |
|
siva
2011/10/27 21:35:07
shouldn't this be _exception === null ?
srdjan
2011/10/27 21:50:11
Good catch! Obviously we do not have enough tests
|
| } |
| void then(void onComplete(T value)) { |
| @@ -119,7 +119,7 @@ |
| } |
| void _setException(var exception) { |
| - if (exception == null) { |
| + if (exception === null) { |
| // null is not a legal value for the exception of a Future |
| throw new IllegalArgumentException(null); |
| } |