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

Removed 'Initializer' import and use as it seems to not exist anymore #455

Merged
merged 1 commit into from
Oct 3, 2019
Merged

Removed 'Initializer' import and use as it seems to not exist anymore #455

merged 1 commit into from
Oct 3, 2019

Conversation

NULLx76
Copy link
Contributor

@NULLx76 NULLx76 commented Oct 2, 2019

This was causing compilation issues for me which are now fixed.

This was causing compilation issues which are now fixed for me
@jdonszelmann
Copy link

jdonszelmann commented Oct 2, 2019

seems to also suffer from rust-lang/rust#64964. CI doesn't compile it but the previous nightly version from september 28th does

Copy link
Collaborator

@Ralith Ralith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Looks like the new futures-preview made it unstable in the same vein as io::Read.

@NULLx76
Copy link
Contributor Author

NULLx76 commented Oct 3, 2019

The compiler bug seems to be fixed with the release of nightly-2019-10-02 so the code compiles correctly now. The CI can now be ran again if necessary.

@djc
Copy link
Member

djc commented Oct 3, 2019

@djc djc merged commit 4560e63 into quinn-rs:master Oct 3, 2019
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