Chromium Code Reviews| Index: lib/core/errors.dart |
| diff --git a/lib/core/errors.dart b/lib/core/errors.dart |
| index c5164262b001829302b756feef6d9150f6e55322..5984e1249c12098aff23ded917a9e3f39e59eac8 100644 |
| --- a/lib/core/errors.dart |
| +++ b/lib/core/errors.dart |
| @@ -44,9 +44,9 @@ class CastError implements Error { |
| /** |
| * Exception thrown because of an index outside of the valid range. |
| * |
| - * Temporarily extends [Exception] for backwards compatiblity. |
| + * Temporarily implements [IndexOutOfRangeException] for backwards compatiblity. |
| */ |
| -class RangeError extends ArgumentError implements Exception { |
| +class RangeError extends ArgumentError implements IndexOutOfRangeException { |
| // TODO(lrn): This constructor should be called only with string values. |
| // It currently isn't in all cases. |
| /** Create a new [RangeError] with the given [message]. */ |
| @@ -63,11 +63,9 @@ class RangeError extends ArgumentError implements Exception { |
| * |
| * This class allows code throwing the old [IndexOutOfRangeException] to |
| * work until they change to the new [RangeError] name. |
| - * Code **catching** IndexOutOfRangeException will fail to catch |
| - * the [RangeError] objects that are now thrown. |
| */ |
| -class IndexOutOfRangeException extends ArgumentError { |
| - IndexOutOfRangeException(var message) : super(message); |
| +interface IndexOutOfRangeException implements Exception default RangeError { |
|
Søren Gjesse
2012/11/02 08:14:57
interface still works?
Lasse Reichstein Nielsen
2012/11/02 08:20:29
Yes. Redirecting factory constructors don't work y
|
| + IndexOutOfRangeException(var message); |
| } |