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

Unified Diff: lib/core/expect.dart

Issue 11238035: Make isEmpty a getter. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Update status file with co19 issue number. Created 8 years, 2 months 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 | « lib/core/collection.dart ('k') | lib/core/future.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/core/expect.dart
diff --git a/lib/core/expect.dart b/lib/core/expect.dart
index 2b87297d315d0484a9ee354d8b7c077f2727c9d1..bdd24eb1f29ef28588d43bc4caff3f72b131a821 100644
--- a/lib/core/expect.dart
+++ b/lib/core/expect.dart
@@ -227,12 +227,12 @@ class Expect {
final extraSet = new Set.from(actual);
extraSet.removeAll(expected);
- if (extraSet.isEmpty() && missingSet.isEmpty()) return;
+ if (extraSet.isEmpty && missingSet.isEmpty) return;
String msg = _getMessage(reason);
StringBuffer sb = new StringBuffer("Expect.setEquals($msg) fails");
// Report any missing items.
- if (!missingSet.isEmpty()) {
+ if (!missingSet.isEmpty) {
sb.add('\nExpected collection does not contain: ');
}
@@ -241,7 +241,7 @@ class Expect {
}
// Report any extra items.
- if (!extraSet.isEmpty()) {
+ if (!extraSet.isEmpty) {
sb.add('\nExpected collection should not contain: ');
}
« no previous file with comments | « lib/core/collection.dart ('k') | lib/core/future.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698