-
Notifications
You must be signed in to change notification settings - Fork 13.3k
rustdoc: Update doc comment for splitn_mut to include mutable in the … #102326
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Hey! It looks like you've submitted a new PR for the library teams! If this PR contains changes to any Examples of
|
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @thomcc (or someone else) soon. Please see the contribution instructions for more information. |
Thanks. @bors r+ rollup |
… r=thomcc rustdoc: Update doc comment for splitn_mut to include mutable in the … The doc comment for [splitn](https://github.com/rust-lang/rust/blob/master/library/core/src/slice/mod.rs#L2051:L2056) is the exact same as the comment for [splitn_mut](https://github.com/rust-lang/rust/blob/master/library/core/src/slice/mod.rs#L2079:L2084). The doc comment for `splitn_mut` should instead say it's working on a mutable subslice.
Rollup of 6 pull requests Successful merges: - rust-lang#102283 (Improve code example for Option::unwrap_or_default) - rust-lang#102319 (rustdoc: merge CSS `table` rules into `.docblock`) - rust-lang#102321 ( Rustdoc-Json: List impls for primitives) - rust-lang#102322 (Document that Display automatically implements ToString) - rust-lang#102325 (rustdoc: give `.line-number` / `.line-numbers` meaningful names) - rust-lang#102326 (rustdoc: Update doc comment for splitn_mut to include mutable in the …) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
The doc comment for splitn is the exact same as the comment for splitn_mut. The doc comment for
splitn_mut
should instead say it's working on a mutable subslice.