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

Vector destructuring from tail, take 2 #5122

Closed
wants to merge 2 commits into from

Conversation

sanxiyn
Copy link
Member

@sanxiyn sanxiyn commented Feb 26, 2013

Incorporated @nikomatsakis's comments from #4748.

Fix #4635.

@sanxiyn
Copy link
Member Author

sanxiyn commented Feb 27, 2013

core::util::unreachable() failed on Windows, so changed to ::core::util::unreachable(). By the way, does anyone know why?

@graydon
Copy link
Contributor

graydon commented Feb 27, 2013

No, that sounds deeply strange. Can you open a bug?

@nikomatsakis
Copy link
Contributor

@sanxiyn I'd like to see this patch land, is it possible for you to rebase and add in the suggestions?

@sanxiyn
Copy link
Member Author

sanxiyn commented Mar 11, 2013

Rebased, fixed a copy, added a test.

bors added a commit that referenced this pull request Mar 11, 2013
@bors bors closed this Mar 11, 2013
@graydon
Copy link
Contributor

graydon commented Mar 11, 2013

Excellent, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants