Skip to content

Additional docs for Vec, String, and slice trait impls #39886

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Feb 25, 2017

Conversation

mbrubeck
Copy link
Contributor

/// every operation, which would lead to `O(n^2)` running time when building an `n`-byte string by
/// repeated concatenation.
///
/// The string on the left is only borrowed; its contents are copied into the returned `String`.
Copy link
Member

Choose a reason for hiding this comment

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

“left-hand operand” may be a better way to name… operand on the left.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops, also this sentence is supposed to be about the right-hand operand anyway.

@steveklabnik
Copy link
Member

@bors: r+ rollup

looks great!

@bors
Copy link
Collaborator

bors commented Feb 21, 2017

📌 Commit b2ac1c9 has been approved by steveklabnik

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Feb 22, 2017
Additional docs for Vec, String, and slice trait impls

r? @steveklabnik
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 23, 2017
Additional docs for Vec, String, and slice trait impls

r? @steveklabnik
bors added a commit that referenced this pull request Feb 23, 2017
Rollup of 11 pull requests

- Successful merges: #39859, #39886, #39892, #39903, #39914, #39918, #39988, #40010, #40027, #40030, #40035
- Failed merges:
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Feb 23, 2017
Additional docs for Vec, String, and slice trait impls

r? @steveklabnik
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 23, 2017
Additional docs for Vec, String, and slice trait impls

r? @steveklabnik
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Feb 24, 2017
Additional docs for Vec, String, and slice trait impls

r? @steveklabnik
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 24, 2017
Additional docs for Vec, String, and slice trait impls

r? @steveklabnik
bors added a commit that referenced this pull request Feb 24, 2017
Rollup of 17 pull requests

- Successful merges: #39777, #39815, #39845, #39886, #39892, #39903, #39905, #39914, #39927, #39940, #40010, #40030, #40048, #40050, #40052, #40060, #40071
- Failed merges:
bors added a commit that referenced this pull request Feb 25, 2017
Rollup of 11 pull requests

- Successful merges: #39777, #39815, #39845, #39886, #39940, #40010, #40030, #40048, #40050, #40052, #40071
- Failed merges:
@bors bors merged commit b2ac1c9 into rust-lang:master Feb 25, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants