Skip to content

Rollup of 8 pull requests - #36042

Closed
Manishearth wants to merge 20 commits into
rust-lang:masterfrom
Manishearth:rollup
Closed

Rollup of 8 pull requests#36042
Manishearth wants to merge 20 commits into
rust-lang:masterfrom
Manishearth:rollup

Conversation

Stebalienand others added 20 commits August 20, 2016 15:20
Doing otherwise would break traits like `ToString`.
This pull request adds a module-level example of how all the range operators work. It also slims down struct-level examples in lieu of a link to module examples.
add feature for inclusive_range_syntax
fix incorrectly closed code fences
They're the same thing but it's better to keep the terminology consistent.
r? @steveklabnik
add examples that lift `<<` and `>>` to a trivial struct
replace `Scalar` structs with struct tuples
add `fn main` wrappers to enable Rust Playground "Run" button
The affected API apparently had changed with commit d59accf.
…les, r=steveklabnik
refactor range examples
This pull request adds a module-level example of how all the range operators work. It also slims down struct-level examples in lieu of a link to module examples.
Clarify/fix formatting docs concerning fmt::Result/fmt::Error
1. `fmt::Result` != `io::Result<()>`
2. Formatters should only propagate errors, not return their own.
Confusion on reddit: https://www.reddit.com/r/rust/comments/4yorxr/is_implt_tostring_for_t_where_t_display_sized_a/
…klabnik
add evocative examples for `Shl` and `Shr`
r? @steveklabnik
…klabnik
add links to interesting items in `std::ptr` documentation
r? @steveklabnik
…steveklabnik
Updated code sample in chapter on syntax extensions.
The affected API apparently had changed with commit d59accf.
---
Further more I had to add
```toml
[lib]
name = "roman_numerals"
crate-type = ["dylib"]
```
to `Cargo.toml` as I otherwise got this compiler error (despite `#![crate_type="dylib"]`):
[E0457]: plugin `roman_numerals` only found in rlib format, but must be available in dylib format
Might be worth adding a note about that?
Rename {int,uint} methods to {isize,usize}.
None
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @brson

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

@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors r+ p=10

@bors

bors commented Aug 27, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit 90df078 has been approved by Manishearth

@bors

bors commented Aug 27, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 90df078 with merge 7e8ccd7...

@bors

bors commented Aug 27, 2016

Copy link
Copy Markdown
Collaborator

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

@alexcrichton

Copy link
Copy Markdown
Member

Seems old now I think? (closing)

@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.

12 participants

@Manishearth@rust-highfive@bors@alexcrichton@brson@Centril@Stebalien@matthew-piziak@GuillaumeGomez@ahmedcharles@regexident@frewsxcv