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

Unified Diff: utils/pub/utils.dart

Issue 11410086: Use iterator, moveNext(), current. (Closed) Base URL: https://dart.googlecode.com/svn/experimental/lib_v2/dart
Patch Set: Address comments. 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
Index: utils/pub/utils.dart
diff --git a/utils/pub/utils.dart b/utils/pub/utils.dart
index d26504d9846e3f4e176d9bc849030c742086c988..1b4d93fce182df2dc4c485fac23656136d0f83cf 100644
--- a/utils/pub/utils.dart
+++ b/utils/pub/utils.dart
@@ -74,10 +74,11 @@ List flatten(List nested) {
* Asserts that [iter] contains only one element, and returns it.
*/
only(Iterable iter) {
- var iterator = iter.iterator();
- assert(iterator.hasNext);
- var obj = iterator.next();
- assert(!iterator.hasNext);
+ var iterator = iter.iterator;
+ var currentIsValid = iterator.moveNext();
+ assert(currentIsValid);
+ var obj = iterator.current;
+ assert(!iterator.moveNext());
return obj;
}

Powered by Google App Engine
This is Rietveld 408576698