Skip to content

Conversation

Dylan-DPC-zz
Copy link

Successful merges:

Failed merges:

r? @ghost

sapir and others added 14 commits April 10, 2020 03:52
Use `ObligationClause::dummy()` when appropriate or replace
`hir::DUMMY_HIR_ID` by `hir::CRATE_HIR_ID`, as used in
`ObligationClause::dummy()`.
…ing, r=matthewjasper

Fix warning for unused variables in or pattern (issue rust-lang#67691)

Is this a good way to fix it?

Also, the tests fail, the "fixed" code output says `{ i, j }` instead of `{ i, j: _ }`, how can I fix that?
…=eddyb

Pass the `PlaceElem::Index` local to `visit_local`

Fixes rust-lang#71008

cc @rust-lang/wg-mir-opt

r? @spastorino
…-clause, r=eddyb

Remove some usage of `DUMMY_HIR_ID`

Use `ObligationClause::dummy()` when appropriate or replace `hir::DUMMY_HIR_ID`by `hir::CRATE_HIR_ID`, as used in `ObligationClause::dummy()`.
@Dylan-DPC-zz
Copy link
Author

@bors p=4 r+ rollup=never

@bors
Copy link
Collaborator

bors commented Apr 12, 2020

📌 Commit 69862b7 has been approved by Dylan-DPC

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Apr 12, 2020
@bors
Copy link
Collaborator

bors commented Apr 12, 2020

⌛ Testing commit 69862b7 with merge 9fed360...

@Dylan-DPC-zz Dylan-DPC-zz added the rollup A PR which is a rollup label Apr 12, 2020
@bors
Copy link
Collaborator

bors commented Apr 13, 2020

☀️ Test successful - checks-azure
Approved by: Dylan-DPC
Pushing 9fed360 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Apr 13, 2020
@bors bors merged commit 9fed360 into rust-lang:master Apr 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. rollup A PR which is a rollup S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants