about summary refs log tree commit diff
diff options
context:
space:
mode:
authorCorey Farwell <coreyf@rwell.org>2016-07-22 21:43:59 -0400
committerCorey Farwell <coreyf@rwell.org>2016-07-23 09:08:45 -0400
commit1e0043eb6c445fb96981b6d46dae4c93af4fbda3 (patch)
treeadc42921d3226effac2b4ecdded985ce3777fbb1
parentad264f7f39b6113fb87a489d72d31abbc03ab148 (diff)
downloadrust-1e0043eb6c445fb96981b6d46dae4c93af4fbda3.tar.gz
rust-1e0043eb6c445fb96981b6d46dae4c93af4fbda3.zip
Fix incorrect 'memory leak' example for `Vec::set_len`.
Example was written in https://github.com/rust-lang/rust/pull/34911

Issue was brought up in this comment:

https://github.com/rust-lang/rust/commit/a005b2cd2ac679da7393e537aa05e2b7d32d36d5#commitcomment-18346958
-rw-r--r--src/libcollections/vec.rs7
1 files changed, 4 insertions, 3 deletions
diff --git a/src/libcollections/vec.rs b/src/libcollections/vec.rs
index 2d77b38879b..967baccd274 100644
--- a/src/libcollections/vec.rs
+++ b/src/libcollections/vec.rs
@@ -593,11 +593,12 @@ impl<T> Vec<T> {
     /// ```
     ///
     /// In this example, there is a memory leak since the memory locations
-    /// owned by the vector were not freed prior to the `set_len` call:
+    /// owned by the inner vectors were not freed prior to the `set_len` call:
     ///
     /// ```
-    /// let mut vec = vec!['r', 'u', 's', 't'];
-    ///
+    /// let mut vec = vec![vec![1, 0, 0],
+    ///                    vec![0, 1, 0],
+    ///                    vec![0, 0, 1]];
     /// unsafe {
     ///     vec.set_len(0);
     /// }