Skip to content

Improve docs for std::vec #19943

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

Merged
merged 1 commit into from
Dec 17, 2014
Merged

Improve docs for std::vec #19943

merged 1 commit into from
Dec 17, 2014

Conversation

steveklabnik
Copy link
Member

Random improvements to the std::vec docs.

@@ -10,7 +10,41 @@

//! A growable list type, written `Vec<T>` but pronounced 'vector.'
//!
//! Vectors own the items of the list.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe my smug programmer is showing but this feels weird to assert. (also vaguely ambiguous if it's a Vec<&T>)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was trying to incorporate the previous documentation, but yeah, maybe this is just redundant.

@tbu-
Copy link
Contributor

tbu- commented Dec 17, 2014

@gankro If @steveklabnik has run the doctests, this would be suitable for rollup.

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Dec 17, 2014
Random improvements to the `std::vec` docs.
@bors bors merged commit 18c420e into rust-lang:master Dec 17, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants