diff options
| author | Jacob Pratt <jacob@jhpratt.dev> | 2024-12-20 01:36:45 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-12-20 01:36:45 -0500 |
| commit | 3cdc3b7e0b516791f4a95eac8a577c37fc75b6ca (patch) | |
| tree | 3a98a73d2f308851c941413f2530a1dc162b859d /library/alloc/src/vec | |
| parent | 214587c89d527dd0ccbe1f2150c737d3bdee67b0 (diff) | |
| parent | 07ba1fdd59cbf563536c40d82c5690008d67937f (diff) | |
| download | rust-3cdc3b7e0b516791f4a95eac8a577c37fc75b6ca.tar.gz rust-3cdc3b7e0b516791f4a95eac8a577c37fc75b6ca.zip | |
Rollup merge of #126118 - jan-ferdinand:docs_for_vec_set_len, r=the8472
docs: Mention `spare_capacity_mut()` in `Vec::set_len` I recently went down a small rabbit hole when trying to identify safe use of `Vec::set_len`. The solution was `Vec::spare_capacity_mut`. I think the docs on `Vec::set_len` benefit from mentioning this method. A possible counter-argument could be that the [clippy lint `uninit_vec`](https://rust-lang.github.io/rust-clippy/master/index.html#/uninit_vec) already nudges people in the right direction. However, I think a working example on `Vec::set_len` is still beneficial. Happy to hear your thoughts on the matter. :blush:
Diffstat (limited to 'library/alloc/src/vec')
| -rw-r--r-- | library/alloc/src/vec/mod.rs | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/library/alloc/src/vec/mod.rs b/library/alloc/src/vec/mod.rs index 5f4b85b58a9..55496005f40 100644 --- a/library/alloc/src/vec/mod.rs +++ b/library/alloc/src/vec/mod.rs @@ -1824,7 +1824,10 @@ impl<T, A: Allocator> Vec<T, A> { /// /// # Examples /// - /// This method can be useful for situations in which the vector + /// See [`spare_capacity_mut()`] for an example with safe + /// initialization of capacity elements and use of this method. + /// + /// `set_len()` can be useful for situations in which the vector /// is serving as a buffer for other code, particularly over FFI: /// /// ```no_run @@ -1884,6 +1887,8 @@ impl<T, A: Allocator> Vec<T, A> { /// /// Normally, here, one would use [`clear`] instead to correctly drop /// the contents and thus not leak memory. + /// + /// [`spare_capacity_mut()`]: Vec::spare_capacity_mut #[inline] #[stable(feature = "rust1", since = "1.0.0")] pub unsafe fn set_len(&mut self, new_len: usize) { |
