Use iterator, moveNext(), current.

sdk/lib/core/set.dart

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * This class is the public interface of a set. A set is a collection
  * without duplicates.
  */
 abstract class Set<E> extends Collection<E> {
   factory Set() => new _HashSetImpl<E>();
@@ skipping 166 matching lines @@
   }
 
   bool get isEmpty {
     return _backingMap.isEmpty;
   }
 
   int get length {
     return _backingMap.length;
   }
 
-  Iterator<E> iterator() {
-    return new _HashSetIterator<E>(this);
-  }
+  Iterator<E> get iterator => new _HashSetIterator<E>(this);
 
   String toString() {
     return Collections.collectionToString(this);
   }
 
   // The map backing this set. The associations in this map are all
   // of the form element -> element. If a value is not in the map,
   // then it is not in the set.
   _HashMapImpl<E, E> _backingMap;
 }
 
 class _HashSetIterator<E> implements Iterator<E> {
 
-  // TODO(4504458): Replace set_ with set.
-  _HashSetIterator(_HashSetImpl<E> set_)
-    : _nextValidIndex = -1,
-      _entries = set_._backingMap._keys {
-    _advance();
+  _HashSetIterator(_HashSetImpl<E> set)
+    : _keysIterator = set._backingMap._keys.iterator;
+
+  E get current {
+    var result = _keysIterator.current;
+    if (identical(result, _HashMapImpl._DELETED_KEY)) {
+      // TODO(floitsch): improve the error reporting.
+      throw new StateError("Concurrent modification.");
+    }
+    return result;
   }
 
-  bool get hasNext {
-    if (_nextValidIndex >= _entries.length) return false;
-    if (identical(_entries[_nextValidIndex], _HashMapImpl._DELETED_KEY)) {
-      // This happens in case the set was modified in the meantime.
-      // A modification on the set may make this iterator misbehave,
-      // but we should never return the sentinel.
-      _advance();
-    }
-    return _nextValidIndex < _entries.length;
+  bool moveNext() {
+    bool result;
+    do {
+      result = _keysIterator.moveNext();
+    } while (result &&
+             (_keysIterator.current == null ||
+              identical(_keysIterator.current, _HashMapImpl._DELETED_KEY)));
+    return result;
   }
 
-  E next() {
-    if (!hasNext) {
-      throw new StateError("No more elements");
-    }
-    E res = _entries[_nextValidIndex];
-    _advance();
-    return res;
-  }
-
-  void _advance() {
-    int length = _entries.length;
-    var entry;
-    final deletedKey = _HashMapImpl._DELETED_KEY;
-    do {
-      if (++_nextValidIndex >= length) break;
-      entry = _entries[_nextValidIndex];
-    } while ((entry == null) || identical(entry, deletedKey));
-  }
-
-  // The entries in the set. May contain null or the sentinel value.
-  List<E> _entries;
-
-  // The next valid index in [_entries] or the length of [entries_].
-  // If it is the length of [_entries], calling [hasNext] on the
-  // iterator will return false.
-  int _nextValidIndex;
+  Iterator _keysIterator;
 }