Skip to content

Conversation

@GuillaumeGomez
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented Jul 14, 2016

📌 Commit 3aad6ab has been approved by GuillaumeGomez

@rust-highfive
Copy link
Contributor

r? @Manishearth

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

@bors
Copy link
Collaborator

bors commented Jul 14, 2016

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

@bors
Copy link
Collaborator

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
Collaborator

bors commented Jul 15, 2016

📌 Commit 8959374 has been approved by GuillaumeGomez

@bors
Copy link
Collaborator

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