Skip to content

Fix issue with lifetime elision#1

Merged
twittner merged 1 commit intoparitytech:masterfrom
crlf0710:patch-1
Aug 19, 2019
Merged

Fix issue with lifetime elision#1
twittner merged 1 commit intoparitytech:masterfrom
crlf0710:patch-1

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