Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update check::cast::pointer_kind logic to new rustc #46973

Merged
merged 2 commits into from
Dec 26, 2017

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented Dec 23, 2017

Make the match exhaustive, adding handling for anonymous types and
tuple coercions on the way.

Also, exit early when type errors are detected, to avoid error cascades
and the like.

Fixes #33690.
Fixes #46365.
Fixes #46880.

Make the match exhaustive, adding handling for anonymous types and
tuple coercions on the way.

Also, exit early when type errors are detected, to avoid error cascades
and the like.
@arielb1 arielb1 added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Dec 23, 2017
@rust-highfive
Copy link
Collaborator

r? @estebank

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

@arielb1
Copy link
Contributor Author

arielb1 commented Dec 23, 2017

beta-nominating because regressioin

These can happen if prior errors disable defaulting.

Fixes rust-lang#43825.
@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 24, 2017
@estebank
Copy link
Contributor

@bors r+ p=1

@bors
Copy link
Contributor

bors commented Dec 25, 2017

📌 Commit 6aca330 has been approved by estebank

@bors
Copy link
Contributor

bors commented Dec 25, 2017

⌛ Testing commit 6aca330 with merge f6627b9b73f6160edec871ff21afd05ffaf1fff9...

@bors
Copy link
Contributor

bors commented Dec 25, 2017

💔 Test failed - status-travis

@kennytm
Copy link
Member

kennytm commented Dec 25, 2017

@bors retry #47002

@bors
Copy link
Contributor

bors commented Dec 25, 2017

⌛ Testing commit 6aca330 with merge 1dc9c64e04f149a00a83898868e2e6234c7ac04b...

@bors
Copy link
Contributor

bors commented Dec 25, 2017

💔 Test failed - status-travis

@bors
Copy link
Contributor

bors commented Dec 25, 2017

⌛ Testing commit 6aca330 with merge ebddfcb...

bors added a commit that referenced this pull request Dec 25, 2017
Update check::cast::pointer_kind logic to new rustc

Make the match exhaustive, adding handling for anonymous types and
tuple coercions on the way.

Also, exit early when type errors are detected, to avoid error cascades
and the like.

Fixes #33690.
Fixes #46365.
Fixes #46880.
@bors
Copy link
Contributor

bors commented Dec 26, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: estebank
Pushing ebddfcb to master...

@bors bors merged commit 6aca330 into rust-lang:master Dec 26, 2017
@nagisa nagisa added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Jan 7, 2018
@alexcrichton
Copy link
Member

Unfortunately we forgot to backport to 1.23.0 (sorry about that!) so removing beta tags

@alexcrichton alexcrichton removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jan 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants