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

Unified Diff: sdk/lib/collection/set.dart

Issue 12212211: Reapply "New implementation of {,Linked}Hash{Set,Map}." (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fix breaks in pub. Created 7 years, 10 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 | « sdk/lib/collection/map.dart ('k') | sdk/lib/core/map.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sdk/lib/collection/set.dart
diff --git a/sdk/lib/collection/set.dart b/sdk/lib/collection/set.dart
deleted file mode 100644
index 9363676e867eccee06f9a299397fb50a0c97806d..0000000000000000000000000000000000000000
--- a/sdk/lib/collection/set.dart
+++ /dev/null
@@ -1,110 +0,0 @@
-// Copyright (c) 2013, 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.
-
-part of dart.collection;
-
-
-
-class HashSet<E> extends Collection<E> implements Set<E> {
- // 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.
- final _HashMapImpl<E, E> _backingMap;
-
- HashSet() : _backingMap = new _HashMapImpl<E, E>();
-
- /**
- * Creates a [Set] that contains all elements of [other].
- */
- factory HashSet.from(Iterable<E> other) {
- Set<E> set = new HashSet<E>();
- for (final e in other) {
- set.add(e);
- }
- return set;
- }
-
- void clear() {
- _backingMap.clear();
- }
-
- void add(E value) {
- _backingMap[value] = value;
- }
-
- bool remove(Object value) {
- if (!_backingMap.containsKey(value)) return false;
- _backingMap.remove(value);
- return true;
- }
-
- bool contains(E value) {
- return _backingMap.containsKey(value);
- }
-
- Set<E> intersection(Collection<E> collection) {
- Set<E> result = new Set<E>();
- collection.forEach((E value) {
- if (contains(value)) result.add(value);
- });
- return result;
- }
-
- bool isSubsetOf(Collection<E> other) {
- return new Set<E>.from(other).containsAll(this);
- }
-
- bool containsAll(Collection<E> collection) {
- return collection.every((E value) {
- return contains(value);
- });
- }
-
- void forEach(void f(E element)) {
- _backingMap.forEach((E key, E value) {
- f(key);
- });
- }
-
- bool get isEmpty {
- return _backingMap.isEmpty;
- }
-
- int get length {
- return _backingMap.length;
- }
-
- Iterator<E> get iterator => new _HashSetIterator<E>(this);
-
- String toString() {
- return Collections.collectionToString(this);
- }
-}
-
-class _HashSetIterator<E> implements Iterator<E> {
-
- _HashSetIterator(HashSet<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 moveNext() {
- bool result;
- do {
- result = _keysIterator.moveNext();
- } while (result &&
- (_keysIterator.current == null ||
- identical(_keysIterator.current, _HashMapImpl._DELETED_KEY)));
- return result;
- }
-
- Iterator _keysIterator;
-}
« no previous file with comments | « sdk/lib/collection/map.dart ('k') | sdk/lib/core/map.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698