Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

correct signature of SecTrieDB::raw_mut #1143

Merged
merged 1 commit into from
May 25, 2016

Conversation

rphmeier
Copy link
Contributor

&mut T is invariant over its type parameter, so we need to specify the trait's lifetime explicitly rather than coerce it to the elided lifetime

&mut T is invariant over its type parameter, so we need to specify the trait's lifetime explicitly rather than coerce it to the elided lifetime
@rphmeier rphmeier added the A0-pleasereview 🤓 Pull request needs code review. label May 25, 2016
@arkpar
Copy link
Collaborator

arkpar commented May 25, 2016

care to fix SecTrieDB::raw as well?

@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels May 25, 2016
@NikVolf NikVolf added A7-looksgoodcantmerge 🙄 Pull request is reviewed well, but cannot be merged due to conflicts. and removed A8-looksgood 🦄 Pull request is reviewed well. labels May 25, 2016
@NikVolf
Copy link
Contributor

NikVolf commented May 25, 2016

no integration tests were run

@arkpar arkpar closed this May 25, 2016
@arkpar arkpar reopened this May 25, 2016
@NikVolf NikVolf added A8-looksgood 🦄 Pull request is reviewed well. and removed A7-looksgoodcantmerge 🙄 Pull request is reviewed well, but cannot be merged due to conflicts. labels May 25, 2016
@arkpar arkpar merged commit 32298ca into openethereum:master May 25, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants