Skip to content

Rollup of 12 pull requests - #34671

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

Rollup of 12 pull requests#34671
steveklabnik wants to merge 33 commits into
rust-lang:masterfrom
steveklabnik:rollup

Conversation

durkaand others added 30 commits April 27, 2016 23:29
I made the "tuple structs are useless" editorializing a bit weaker and moved it to the end. Feel free to overrule me on that.
I also added an example of how to unpack a tuple struct with dot notation, because it came up on IRC.
`braced_empty_structs` is stable now, so I updated the example for unit-like structs to use that syntax. Should we show both ways?
cc @ubsan
r? @steveklabnik or @GuillameGomez
This adds short summaries to all num consts.
updated proper link of cargo doc that contains details about list of options available in semantic versioninig for the dependencies section in Cargo.toml
Add a space between the comma and j in (i, j) to make it look nice.
update documentation of tuple/unit structs
I made the "tuple structs are useless" editorializing a bit weaker and moved it to the end. Feel free to overrule me on that.
I also added an example of how to unpack a tuple struct with dot notation, because it came up on IRC.
`braced_empty_structs` is stable now, so I updated the example for unit-like structs to use that syntax. Should we show both ways?
cc @ubsan
r? @steveklabnik or @GuillaumeGomez
Add/improve num const docs
This adds short summaries to all num consts.
r? @steveklabnik
…, r=GuillaumeGomez
Add doc examples for `io::Error::from_raw_os_error`.
None
update cargo doc link
updated proper link of cargo doc that contains details about list of options available in semantic versioninig for the dependencies section in Cargo.toml
… r=steveklabnik
Clarifies the meaning of the external mutability.
Fix spacing in for loop enumeration example
Add a space between the comma and j in (i, j) to make it look nice.
This addresses my recent issue rust-lang#34624.
😀
@steveklabnik

Copy link
Copy Markdown
ContributorAuthor

@bors: r+ p=1

@bors

bors commented Jul 5, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit ef8ecf2 has been approved by steveklabnik

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @Manishearth

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

@bors

bors commented Jul 5, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit ef8ecf2 with merge 1f7b8d8...

@bors

bors commented Jul 5, 2016

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-win-msvc-64-cargotest

@steveklabnik

Copy link
Copy Markdown
ContributorAuthor

@bors: retry

@bors

bors commented Jul 6, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit ef8ecf2 with merge e66a04c...

@bors

bors commented Jul 6, 2016

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-mac-32-opt

@GuillaumeGomez

Copy link
Copy Markdown
Member

@steveklabnik: It seems like a legit error.

@steveklabnik

Copy link
Copy Markdown
ContributorAuthor

Yeah, I'm afraid so this time.

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

15 participants

@steveklabnik@bors@rust-highfive@GuillaumeGomez@Centril@Manishearth@durka@ollie27@llogiq@GlenDC@frewsxcv@hhnr@GuillaumeRochat@sylvestre@tshepang