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

Unified Diff: sdk/lib/core/iterable.dart

Issue 451633003: Make Iterable.toSet say that it returns a Set with the same equality as the Iterable's contains met… (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 4 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/splay_tree.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sdk/lib/core/iterable.dart
diff --git a/sdk/lib/core/iterable.dart b/sdk/lib/core/iterable.dart
index 6d10da9fa125fbe1b17cc9329c0bcf4544baa6c9..717478b5464b02fef21c821248a871b2cf9a385a 100644
--- a/sdk/lib/core/iterable.dart
+++ b/sdk/lib/core/iterable.dart
@@ -174,11 +174,13 @@ abstract class Iterable<E> {
/**
* Creates a [Set] containing the same elements as this iterable.
*
- * The returned `Set` will have the same `Set.length`
- * as the `length` of this iterable,
- * and its `Set.contains` will return the same result
+ * The returned `Set` will have a `Set.contains` that return the same result
floitsch 2014/10/02 12:04:17 Remove this sentence. We don't want to give guaran
Lasse Reichstein Nielsen 2014/10/06 10:54:58 Done.
* as the `contains` of this iterable.
- * The order of the elements may be different.
+ * The set may contain fewer elements than the iterable,
+ * if the iterable contains the same element (or equal elements) more than
+ * once.
+ * The order of the elements in the set is not guaranteed to be the same
+ * as for the iterable.
*/
Set<E> toSet();
« no previous file with comments | « sdk/lib/collection/splay_tree.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698