Do not suggest impl traits as type arguments

This commit is contained in:
Yuki Okushi 2021-06-16 06:32:22 +09:00
parent 3354a44d2f
commit 14e92d7116
No known key found for this signature in database
GPG Key ID: DABA5B072961C18A
7 changed files with 71 additions and 16 deletions

View File

@ -753,23 +753,11 @@ pub fn emit_inference_failure_err(
if let (UnderspecifiedArgKind::Const { .. }, Some(parent_data)) =
(&arg_data.kind, &arg_data.parent)
{
let has_impl_trait =
self.tcx.generics_of(parent_data.def_id).params.iter().any(|param| {
matches!(
param.kind,
ty::GenericParamDefKind::Type {
synthetic: Some(
hir::SyntheticTyParamKind::ImplTrait
| hir::SyntheticTyParamKind::FromAttr,
),
..
}
)
});
// (#83606): Do not emit a suggestion if the parent has an `impl Trait`
// as an argument otherwise it will cause the E0282 error.
if !has_impl_trait || self.tcx.features().explicit_generic_args_with_impl_trait {
if !self.tcx.generics_of(parent_data.def_id).has_impl_trait()
|| self.tcx.features().explicit_generic_args_with_impl_trait
{
err.span_suggestion_verbose(
span,
"consider specifying the const argument",
@ -814,7 +802,7 @@ fn annotate_method_call(
let borrow = typeck_results.borrow();
if let Some((DefKind::AssocFn, did)) = borrow.type_dependent_def(e.hir_id) {
let generics = self.tcx.generics_of(did);
if !generics.params.is_empty() {
if !generics.params.is_empty() && !generics.has_impl_trait() {
err.span_suggestion_verbose(
segment.ident.span.shrink_to_hi(),
&format!(

View File

@ -198,6 +198,21 @@ pub fn const_param(&'tcx self, param: &ParamConst, tcx: TyCtxt<'tcx>) -> &Generi
_ => bug!("expected const parameter, but found another generic parameter"),
}
}
/// Returns `true` if `params` has `impl Trait`.
pub fn has_impl_trait(&'tcx self) -> bool {
self.params.iter().any(|param| {
matches!(
param.kind,
ty::GenericParamDefKind::Type {
synthetic: Some(
hir::SyntheticTyParamKind::ImplTrait | hir::SyntheticTyParamKind::FromAttr,
),
..
}
)
})
}
}
/// Bounds on generics.

View File

@ -1603,6 +1603,7 @@ fn maybe_report_ambiguity(
let generics = self.tcx.generics_of(*def_id);
if generics.params.iter().any(|p| p.name != kw::SelfUpper)
&& !snippet.ends_with('>')
&& !generics.has_impl_trait()
{
// FIXME: To avoid spurious suggestions in functions where type arguments
// where already supplied, we check the snippet to make sure it doesn't

View File

@ -0,0 +1,9 @@
// Regression test of #86162.
fn foo(x: impl Clone) {}
fn gen<T>() -> T { todo!() }
fn main() {
foo(gen()); //<- Do not suggest `foo::<impl Clone>()`!
//~^ ERROR: type annotations needed
}

View File

@ -0,0 +1,14 @@
error[E0283]: type annotations needed
--> $DIR/issue-86162-1.rs:7:5
|
LL | fn foo(x: impl Clone) {}
| ----- required by this bound in `foo`
...
LL | foo(gen()); //<- Do not suggest `foo::<impl Clone>()`!
| ^^^ cannot infer type for type parameter `impl Clone` declared on the function `foo`
|
= note: cannot satisfy `_: Clone`
error: aborting due to previous error
For more information about this error, try `rustc --explain E0283`.

View File

@ -0,0 +1,14 @@
// Regression test of #86162.
fn gen<T>() -> T { todo!() }
struct Foo;
impl Foo {
fn bar(x: impl Clone) {}
}
fn main() {
Foo::bar(gen()); //<- Do not suggest `Foo::bar::<impl Clone>()`!
//~^ ERROR: type annotations needed
}

View File

@ -0,0 +1,14 @@
error[E0283]: type annotations needed
--> $DIR/issue-86162-2.rs:12:5
|
LL | fn bar(x: impl Clone) {}
| ----- required by this bound in `Foo::bar`
...
LL | Foo::bar(gen()); //<- Do not suggest `Foo::bar::<impl Clone>()`!
| ^^^^^^^^ cannot infer type for type parameter `impl Clone` declared on the associated function `bar`
|
= note: cannot satisfy `_: Clone`
error: aborting due to previous error
For more information about this error, try `rustc --explain E0283`.