Skip to content

Conversation

@rust-highfive
Copy link
Contributor

r? @steveklabnik

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

@GuillaumeGomez
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented Jul 28, 2016

📌 Commit 271d977 has been approved by GuillaumeGomez

@bors
Copy link
Collaborator

bors commented Jul 29, 2016

⌛ Testing commit 271d977 with merge 79c7433...

@bors
Copy link
Collaborator

bors commented Jul 29, 2016

💔 Test failed - auto-win-gnu-32-opt-rustbuild

book/ffi: nullable pointer cleanup

Expand the "nullable pointer optimization" section with a code example. Fixes rust-lang#34250.

I also noticed that many of the examples use the libc crate just for types such as `c_char` and `c_int`, which are now available through `std::os::raw`. I changed the ones that don't need to rely on libc. I'm glad to revert that part of the commit if it's unwanted churn.
…abnik

More intuitive explantion of strings formatting
…omez

Add documentation example for `str::Chars::as_str`.

None
@GuillaumeGomez
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented Jul 29, 2016

📌 Commit 5f5fdf6 has been approved by GuillaumeGomez

@bors
Copy link
Collaborator

bors commented Jul 29, 2016

⌛ Testing commit 5f5fdf6 with merge 5a7773a...

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

- Successful merges: #34258, #34894, #35050, #35062, #35066, #35072, #35087
- Failed merges:
@bors bors merged commit 5f5fdf6 into rust-lang:master Jul 29, 2016
@GuillaumeGomez GuillaumeGomez deleted the rollup branch July 29, 2016 16:19
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
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