Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Rollup of 3 pull requests #41503

Closed
wants to merge 10 commits into from
Closed

Rollup of 3 pull requests #41503

wants to merge 10 commits into from

Conversation

frewsxcv
Copy link
Member

malbarbo and others added 10 commits April 18, 2017 13:50
@malbarbo
Add bootstrap support for android
a42c025
@SimonSapin @mattico
Add Vec::splice and String::splice
2111aff
@mattico
Update splice impl
b85e2e4
@mattico
Improve splice docs and tests
cec00ba
@mattico
Use Vec::splice impl in string::Splice::drop()
c3baa8c
@mattico
Add splice to the unstable book.
7b86ba0
@steveklabnik
use the word 'length' in Vec::len's docs
f852e3f 
Fixes rust-lang#37866
@frewsxcv
Rollup merge of rust-lang#40434 - mattico:splice-update, r=alexcrichton
1ae52ab 
Implement Vec::splice and String::splice (RFC 1432)

RFC: rust-lang/rfcs#1432, tracking issue: rust-lang#32310
A rebase of rust-lang#32355 with a few more tests.

Let me know if you have any ideas for more tests.

cc @SimonSapin
@frewsxcv
Rollup merge of rust-lang#41370 - malbarbo:android-bootstrap, r=alexc…
2d2056e 
…richton

Add bootstrap support for android
@frewsxcv
Rollup merge of rust-lang#41500 - steveklabnik:gh37866, r=frewsxcv
71dfeb1 
use the word 'length' in Vec::len's docs

Fixes rust-lang#37866
@frewsxcv
Copy link
Member Author

@bors r+ p=10

@bors
Copy link
Contributor

bors commented Apr 24, 2017

📌 Commit 71dfeb1 has been approved by frewsxcv

@bors
Copy link
Contributor

bors commented Apr 24, 2017

⌛ Testing commit 71dfeb1 with merge 38c99d2...

bors added a commit that referenced this pull request Apr 24, 2017
@bors
Auto merge of #41503 - frewsxcv:rollup, r=frewsxcv
38c99d2 
Rollup of 3 pull requests

- Successful merges: #40434, #41370, #41500
- Failed merges:
@bors
Copy link
Contributor

bors commented Apr 24, 2017

💔 Test failed - status-appveyor

@frewsxcv frewsxcv mentioned this pull request Apr 24, 2017
Closed
@frewsxcv
Copy link
Member Author

#41511

@frewsxcv frewsxcv closed this Apr 24, 2017
@frewsxcv frewsxcv deleted the rollup branch April 24, 2017 18:31
@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
rollup A PR which is a rollup
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@frewsxcv @bors @Centril @malbarbo @SimonSapin @mattico @steveklabnik