Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(553)

Unified Diff: lib/core/errors.dart

Issue 11365059: Temporarily reintroduce IndexOutOfRangeError matcher in unittests. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | pkg/unittest/lib/src/core_matchers.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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);
}
« no previous file with comments | « no previous file | pkg/unittest/lib/src/core_matchers.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698