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

Fix mistakenly derive hash for struct that contains IncompleteArrayField #1094

Merged
merged 1 commit into from Oct 24, 2017

Conversation

Projects
None yet
5 participants
@photoszzt
Copy link
Contributor

photoszzt commented Oct 24, 2017

Fix #1093

r? @fitzgen or @emilio

@highfive

This comment has been minimized.

Copy link
Collaborator

highfive commented Oct 24, 2017

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!
@emilio

This comment has been minimized.

Copy link
Collaborator

emilio commented Oct 24, 2017

@bors-servo r+

Thanks!

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Oct 24, 2017

📌 Commit fa9b5e3 has been approved by emilio

@highfive highfive assigned emilio and unassigned fitzgen Oct 24, 2017

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Oct 24, 2017

⌛️ Testing commit fa9b5e3 with merge dc1e546...

bors-servo added a commit that referenced this pull request Oct 24, 2017

Auto merge of #1094 - photoszzt:fix_hash_incomplete_arr, r=emilio
Fix mistakenly derive hash for struct that contains IncompleteArrayField

Fix #1093

r? @fitzgen  or @emilio
@emilio

emilio approved these changes Oct 24, 2017

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Oct 24, 2017

☀️ Test successful - status-travis
Approved by: emilio
Pushing dc1e546 to master...

@bors-servo bors-servo merged commit fa9b5e3 into rust-lang:master Oct 24, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.