Fix build on nightly rustc#38
Closed
petrochenkov wants to merge 1 commit intoreem:masterfrom
petrochenkov:master
Closed
Fix build on nightly rustc#38petrochenkov wants to merge 1 commit intoreem:masterfrom petrochenkov:master
petrochenkov wants to merge 1 commit intoreem:masterfrom
petrochenkov:master
Conversation
Owner
|
Is defaulting like this not allowed any more? Is there a way to fix the issue while also maintaining the previous default? |
Owner
|
Ah nvm, I see this is on the impl not the definition. I merged #36 instead since it fixes some other errors, is that sufficient to solve the problem? |
Author
bors
added a commit
to rust-lang/rust
that referenced
this pull request
Jun 1, 2017
Turn sufficiently old compatibility lints into hard errors It's been almost 7 months since #36894 was merged, so it's time to take the next step. [breaking-change], needs crater run. PRs/issues submitted to affected crates: https://github.com/alexcrichton/ctest/pull/17 Sean1708/rusty-cheddar#55 m-r-r/helianto#3 azdle/virgil#1 rust-locale/rust-locale#24 mneumann/acyclic-network-rs#1 reem/rust-typemap#38 cc https://internals.rust-lang.org/t/moving-forward-on-forward-compatibility-lints/4204 cc #34537 #36887 Closes #36886 Closes #36888 Closes #36890 Closes #36891 Closes #36892 r? @nikomatsakis
frewsxcv
added a commit
to frewsxcv/rust
that referenced
this pull request
Jun 1, 2017
Turn sufficiently old compatibility lints into hard errors It's been almost 7 months since rust-lang#36894 was merged, so it's time to take the next step. [breaking-change], needs crater run. PRs/issues submitted to affected crates: https://github.com/alexcrichton/ctest/pull/17 Sean1708/rusty-cheddar#55 m-r-r/helianto#3 azdle/virgil#1 rust-locale/rust-locale#24 mneumann/acyclic-network-rs#1 reem/rust-typemap#38 cc https://internals.rust-lang.org/t/moving-forward-on-forward-compatibility-lints/4204 cc rust-lang#34537 rust-lang#36887 Closes rust-lang#36886 Closes rust-lang#36888 Closes rust-lang#36890 Closes rust-lang#36891 Closes rust-lang#36892 r? @nikomatsakis
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi.
rust-lang/rust#42136 is going to turn some old compatibility warnings in
rustcinto hard errors.crates.iotesting discovered that this crate is affected by one of those errors (default type parameters on impls).This PR updates the code to build successfully with rust-lang/rust#42136