Skip to content

Rollup of 8 pull requests - #33494

Merged
bors merged 15 commits into
rust-lang:masterfrom
Manishearth:rollup
May 8, 2016
Merged

Rollup of 8 pull requests#33494
bors merged 15 commits into
rust-lang:masterfrom
Manishearth:rollup

Conversation

nikomatsakisand others added 8 commits May 3, 2016 06:15
to get more reproducible output.
Fixes: rust-lang#32555
It's pretty confusing and this error triggers in resolve only when "shadowing" a
const, so let's make that clearer.
Adds documentation to Clone, specifying that Copy types should have a trivial Clone impl.
Fixesrust-lang#33416.
If the `-f` option isn't given, GNU tar will use environment variable
`TAPE` first, and next use the compiled-in default, which isn't
necessary `stdout` (it is the tape device `/dev/rst0` under OpenBSD for
example).
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @jroesch

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

@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors r+ p=20

@bors

bors commented May 8, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit cd9efdf has been approved by Manishearth

@bors

bors commented May 8, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit cd9efdf with merge db94e38...

bors added a commit that referenced this pull request May 8, 2016
Rollup of 8 pull requests
- Successful merges: #33369, #33373, #33404, #33406, #33420, #33424, #33426, #33473
- Failed merges:
@bors

bors commented May 8, 2016

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-linux-64-opt-rustbuild

@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors r+

@bors

bors commented May 8, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit 336ea2c has been approved by Manishearth

@bors

bors commented May 8, 2016

Copy link
Copy Markdown
Collaborator

🔒 Merge conflict

…=jntrnr
degrade gracefully with empty spans
In rust-lang#32756, we solved the final test failure, but digging more into it the handling of that scenario could be better. The error was caused by an empty span supplied by the parser representing EOF. This patch checks that we cope more gracefully with such spans:
r? @jonathandturner
rustdoc: use btree map for where clauses
to get more reproducible output.
Fixes: rust-lang#32555
I've looked at the other uses of HashMap in rustdoc, and they seem ok to (i.e. they use `iter()` and related only for constructing a new map, or when the output goes into independent files).
Not sure what the cause of rust-lang#24473 is, it shouldn't be where clauses, but maybe it was also fixed inbetween since May 2015.
… r=GuillaumeGomez
Improve diagnostics for constants being used in irrefutable patterns
It's pretty confusing and this error triggers in resolve only when "shadowing" a const, so let's make that clearer.
r? @steveklabnik
implement RFC 1521
Adds documentation to Clone, specifying that Copy types should have a trivial Clone impl.
Fixesrust-lang#33416.
I tried to use "should" and "must" as defined [here](https://tools.ietf.org/html/rfc2119).
cc @ubsan
make dist: specify the archive file as stdout
If the `-f` option isn't given, GNU tar will use environment variable
`TAPE` first, and next use the compiled-in default, which isn't
necessary `stdout` (it is the tape device `/dev/rst0` under OpenBSD for
example).
@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors r+ force

@bors

bors commented May 8, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit cc98f4c has been approved by Manishearth

@bors

bors commented May 8, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit cc98f4c with merge cae42a4...

bors added a commit that referenced this pull request May 8, 2016
Rollup of 8 pull requests
- Successful merges: #33369, #33373, #33404, #33406, #33420, #33424, #33426, #33473
- Failed merges:
@bors
bors merged commit cc98f4c into rust-lang:masterMay 8, 2016
@CentrilCentril added the rollup A PR which is a rollup label Oct 2, 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.

11 participants

@Manishearth@rust-highfive@bors@jroesch@Centril@nikomatsakis@birkenfeld@gsquire@durka@semarie@sfackler