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 issue with lifetime elision #1

Merged
merged 1 commit into from
Aug 19, 2019
Merged

Conversation

crlf0710
Copy link
Contributor

Hello, according to rust-lang/rust#63376 , the compiler used to accept the current function signature of drain_extensions() by mistake. The compiler will soon fix this (when the above said PR lands), and after that the current code won't compile.

This PR fixes the code. Please note that the latest published version soketto-0.2.2 contains this error too.

Hello, according to rust-lang/rust#63376 , the compiler used to accept the current function signature of `drain_extensions()` by mistake, the compiler will soon fix this (after the above PR lands), and after that the current code won't compile.

This PR fixes the code. Please note that the latest published version soketto-0.2.2 contains this error too.
@twittner twittner merged commit 490bcd7 into paritytech:master Aug 19, 2019
@twittner
Copy link
Contributor

Thank you!

twittner added a commit to twittner/substrate that referenced this pull request Aug 21, 2019
This version fixes an issue with lifetime elision which causes
compilation to fail on recent rustc versions (e.g. 1.39.0-nightly
(bea0372a1 2019-08-20)).

See paritytech/soketto#1 for more information.
gavofyork pushed a commit to paritytech/substrate that referenced this pull request Aug 21, 2019
This version fixes an issue with lifetime elision which causes
compilation to fail on recent rustc versions (e.g. 1.39.0-nightly
(bea0372a1 2019-08-20)).

See paritytech/soketto#1 for more information.
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.

2 participants