Skip to content

Conversation

@rust-highfive
Copy link
Contributor

r? @eddyb

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

@Manishearth
Copy link
Member Author

@bors r+ p=20

@bors
Copy link
Collaborator

bors commented Jul 28, 2016

📌 Commit e9b5f3a has been approved by Manishearth

@bors
Copy link
Collaborator

bors commented Jul 28, 2016

⌛ Testing commit e9b5f3a with merge cc74333...

@bors
Copy link
Collaborator

bors commented Jul 28, 2016

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

@Manishearth
Copy link
Member Author

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 28, 2016

📌 Commit bc283bb has been approved by Manishearth

Make vec::Drain and binary_heap::Drain covariant

I removed all mutable pointers/references, and added covariance tests similar to the ones in rust-lang#32635. It builds and passes the tests, but I noticed that there weren't any tests of Drain's behaviour (at least not in libcollectionstest), so I'm not sure if my changes accidently broke Drain's behaviour. Should I add some tests for that (and if so, what should the tests include)?
resolve: Fix ICE and extra diagnostics happening when unresolved imports are used in patterns

Closes rust-lang#34933

r? @jseyfried
Avoid processing `feature`s on unconfigured crates

Fixes rust-lang#34932, a regression caused by rust-lang#34272.
r? @nrc
…xcrichton

move coerce-match{,-calls} into run-pass-valgrind

Closes rust-lang#21696.
… r=alexcrichton

rustdoc: Fix tuple struct where clause rendering

For tuple structs the where clause comes after the definition.

Fixes rust-lang#34928
…sakis

Use &mut for `CombineFields` in inference relations

Because dropping obligations on the floor makes a mess.

r? @nikomatsakis
@bors
Copy link
Collaborator

bors commented Jul 28, 2016

⌛ Testing commit bc283bb with merge 748ecb1...

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

- Successful merges: #34951, #34963, #34969, #35013, #35037, #35040, #35058
- Failed merges:
@bors bors merged commit bc283bb into rust-lang:master Jul 28, 2016
@eddyb eddyb deleted the rollup branch July 28, 2016 18:25
@bors bors mentioned this pull request Jul 28, 2016
@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.