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.

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

Rollup of 7 pull requests #34819

Merged
merged 14 commits into from
Jul 15, 2016
Merged

Rollup of 7 pull requests #34819

merged 14 commits into from
Jul 15, 2016

Conversation

@GuillaumeGomez
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Contributor

bors commented Jul 14, 2016

📌 Commit 3aad6ab has been approved by GuillaumeGomez

@rust-highfive
Copy link
Collaborator

r? @Manishearth

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

@bors
Copy link
Contributor

bors commented Jul 14, 2016

⌛ Testing commit 3aad6ab with merge 7e82241...

@bors
Copy link
Contributor

bors commented Jul 14, 2016

💔 Test failed - auto-win-msvc-64-opt-rustbuild

frewsxcv and others added 8 commits July 14, 2016 20:32
Use `ptr::{null, null_mut}` instead of `0 as *{const, mut}`
…eveklabnik

Improve arc doc, fixing rust-lang#32905

As issue rust-lang#32905 detailed, I moved part of the module doc to the struct doc, and fixed some small places in the `alloc::arc`.
doc: Mention that writeln! and println! always use LF

Fixes rust-lang#34697

I'm not really satisfied with the wording, but I didn't have a better idea. Suggestions welcome.
…laumeGomez

Add doc example for `std::process::ExitStatus::success`.

None
doc: ffi referenced rust-snappy can not compile

r? @steveklabnik

The referenced code https://github.com/thestinger/rust-snappy can not work. Maybe it's the old rust version? I do not know.
So I try to rewrite these test cases. If it is not what you originally meaning, just ignored it.
Mention where `std::vec` structs originate from.

None
@GuillaumeGomez
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Contributor

bors commented Jul 15, 2016

📌 Commit 8959374 has been approved by GuillaumeGomez

@bors
Copy link
Contributor

bors commented Jul 15, 2016

⌛ Testing commit 8959374 with merge 3cc3ad1...

bors added a commit that referenced this pull request Jul 15, 2016
Rollup of 7 pull requests

- Successful merges: #34456, #34733, #34777, #34794, #34799, #34804, #34818
- Failed merges: #33951
@bors bors merged commit 8959374 into rust-lang:master Jul 15, 2016
@GuillaumeGomez GuillaumeGomez deleted the rollup branch July 15, 2016 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants