Skip to content

mem_categorization: handle type-based paths in variant patterns #42969

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

Merged
merged 1 commit into from
Jun 30, 2017

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented Jun 29, 2017

These can't be used in correct programs, but must be handled in order to
prevent ICEs.

Fixes #42880.

r? @eddyb

@arielb1 arielb1 force-pushed the struct-err branch 4 times, most recently from 6b839c8 to 92abb9d Compare June 29, 2017 17:19
These can't be used in correct programs, but must be handled in order to
prevent ICEs.

Fixes rust-lang#42880.
@eddyb
Copy link
Member

eddyb commented Jun 29, 2017

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 29, 2017

📌 Commit 1ea6813 has been approved by eddyb

@bors
Copy link
Collaborator

bors commented Jun 30, 2017

⌛ Testing commit 1ea6813 with merge 37849a0...

bors added a commit that referenced this pull request Jun 30, 2017
mem_categorization: handle type-based paths in variant patterns

These can't be used in correct programs, but must be handled in order to
prevent ICEs.

Fixes #42880.

r? @eddyb
@shepmaster shepmaster added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jun 30, 2017
@bors
Copy link
Collaborator

bors commented Jun 30, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: eddyb
Pushing 37849a0 to master...

@bors bors merged commit 1ea6813 into rust-lang:master Jun 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

4 participants