about summary refs log tree commit diff
diff options
context:
space:
mode:
authorStepan Koltsov <stepan.koltsov@gmail.com>2015-11-08 05:30:21 +0300
committerStepan Koltsov <stepan.koltsov@gmail.com>2015-11-08 21:27:12 +0300
commitf6f99ce7a3754a8ab052347d2d27bbb65b2e0560 (patch)
tree2486d51818bc624c702ed0b38afe1fd236e07bde
parent01fc81f249cf8d81bbb5f1d6675bfb14fe20afdd (diff)
downloadrust-f6f99ce7a3754a8ab052347d2d27bbb65b2e0560.tar.gz
rust-f6f99ce7a3754a8ab052347d2d27bbb65b2e0560.zip
Explain that size_hint cannot be trusted
Same applies to `len()` function of `ExactSizeIterator` trait.
-rw-r--r--src/libcore/iter.rs23
1 files changed, 23 insertions, 0 deletions
diff --git a/src/libcore/iter.rs b/src/libcore/iter.rs
index e6dae771792..da7d673cd96 100644
--- a/src/libcore/iter.rs
+++ b/src/libcore/iter.rs
@@ -369,6 +369,25 @@ pub trait Iterator {
     /// `None` here means that either there is no known upper bound, or the
     /// upper bound is larger than `usize`.
     ///
+    /// # Implementation notes
+    ///
+    /// It is not enforced that an iterator implementation yields the
+    /// declared number of elements. A buggy iterator may yield less
+    /// than the lower bound or more than the upper bound of elements.
+    ///
+    /// `size_hint()` is primarily intended to be used for optimizations
+    /// such as reserving space for the elements of the iterator, but
+    /// must not be trusted to e.g. omit bounds checks in unsafe code.
+    /// An incorrect implementation of `size_hint()` should not lead to
+    /// memory safety violations.
+    ///
+    /// That said, the implementation should provide a correct
+    /// estimation, because otherwise it would be a violation of the
+    /// trait's protocol.
+    ///
+    /// The default implementation returns `(0, None)` which is correct
+    /// for any iterator.
+    ///
     /// # Examples
     ///
     /// Basic usage:
@@ -2731,7 +2750,11 @@ pub trait ExactSizeIterator: Iterator {
     /// implementation, you can do so. See the [trait-level] docs for an
     /// example.
     ///
+    /// This function has the same safety guarantees as [`size_hint()`]
+    /// function.
+    ///
     /// [trait-level]: trait.ExactSizeIterator.html
+    /// [`size_hint()`]: trait.Iterator.html#method.size_hint
     ///
     /// # Examples
     ///