Skip to content

Rollup of 11 pull requests - #31350

Closed
steveklabnik wants to merge 29 commits into
rust-lang:masterfrom
steveklabnik:rollup
Closed

Rollup of 11 pull requests#31350
steveklabnik wants to merge 29 commits into
rust-lang:masterfrom
steveklabnik:rollup

Conversation

sandeep-dattaand others added 29 commits January 17, 2016 14:08
E0507 can occur when you try to move out of a member of a mutably
borrowed struct, in which case `mem::replace` can help. Mentioning that
here hopefully saves future users a trip to Google.
There is no `Drop` implemented for `Child`, so if it goes out
of scope in Rust-land and gets deallocated, the child process
will continue to exist and execute. If users want a guarantee
that the process has finished running and exited they must
manually use `kill`, `wait`, or `wait_with_output`.
Fixesrust-lang#31289.
Spelling mistake -
`familliar` > `familiar`
Some other shufflings as well:
* Three powerpc triples for Linux have been added recently
* An armv7 linux triple was added recently
* The 64-bit Solaris triple is now mentioned in tier 3
We are currently now also building nightlies for iOS, powerpc triples, and
armv7, but there hasn't been much vetting of the triples themselves so I've left
them in tier 3 for now.
Updated documentation to clarify the difference between `and_then` and `map`. This also explains why we need `and_then` in addition to `map`. Please look at the diff for more information.
r? @alexcrichton
Fixesrust-lang#30459
Fun fact: i wanted to write "Arabic" and "Hebrew" in Arabic and Hebrew, but vim kept doing the copy/paste in the wrong direction.
E0507 can occur when you try to move out of a member of a mutably borrowed struct, in which case `mem::replace` can help. Mentioning that here hopefully saves future users a trip to Google.
…, r=alexcrichton
`Drop` is not implemented for `Child`, so if it goes out of scope in Rust-land and gets deallocated the child process will continue to exist and execute. If users want a guarantee that the process has finished running and exited they must manually use `kill`, `wait`, or `wait_with_output`.
Fixesrust-lang#31289.
r? @steveklabnik
Spelling mistake -
`familliar` > `familiar`
Fixesrust-lang#31334
This is just a quicker fix for this issue; since I'm working on the next draft of the book, I don't want to put a huuuge amount of work into improving it here.
Space character is missed after `let some_usize`
…klabnik
Some other shufflings as well:
* Three powerpc triples for Linux have been added recently
* An armv7 linux triple was added recently
* The 64-bit Solaris triple is now mentioned in tier 3
We are currently now also building nightlies for iOS, powerpc triples, and
armv7, but there hasn't been much vetting of the triples themselves so I've left
them in tier 3 for now.
@steveklabnik

Copy link
Copy Markdown
ContributorAuthor

@bors: r+ p=1

@bors

bors commented Feb 1, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit 814f254 has been approved by steveklabnik

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@bors

bors commented Feb 2, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 814f254 with merge 6d24534...

@bors

bors commented Feb 2, 2016

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-mac-64-opt

@CentrilCentril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollupA PR which is a rollup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

14 participants

@steveklabnik@bors@rust-highfive@alexcrichton@Centril@sandeep-datta@ruuda@oli-obk@mitaa@dirk@prayagverma@reeze@alopatindev@tshepang