Skip to content

Commit

Permalink
Rollup merge of #122915 - fmease:lt-opaq-mismatch-delay-bug, r=compil…
Browse files Browse the repository at this point in the history
…er-errors

Delay a bug if no RPITITs were found

Fixes #122655. See the issue for context.

r? compiler-errors or compiler
  • Loading branch information
matthiaskrgr authored Mar 23, 2024
2 parents 99e34b4 + 807bd98 commit f03326c
Show file tree
Hide file tree
Showing 3 changed files with 74 additions and 21 deletions.
7 changes: 3 additions & 4 deletions compiler/rustc_hir_analysis/src/check/compare_impl_item.rs
Original file line number Diff line number Diff line change
Expand Up @@ -639,10 +639,9 @@ pub(super) fn collect_return_position_impl_trait_in_trait_tys<'tcx>(
}
}

if !unnormalized_trait_sig.output().references_error() {
debug_assert!(
!collector.types.is_empty(),
"expect >0 RPITITs in call to `collect_return_position_impl_trait_in_trait_tys`"
if !unnormalized_trait_sig.output().references_error() && collector.types.is_empty() {
tcx.dcx().delayed_bug(
"expect >0 RPITITs in call to `collect_return_position_impl_trait_in_trait_tys`",
);
}

Expand Down
17 changes: 15 additions & 2 deletions tests/ui/impl-trait/in-trait/opaque-and-lifetime-mismatch.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
struct Wrapper<'rom>(T);
//~^ ERROR cannot find type `T` in this scope
struct Wrapper<'rom>(&'rom ());

trait Foo {
fn bar() -> Wrapper<impl Sized>;
Expand All @@ -15,4 +14,18 @@ impl Foo for () {
}
}

trait Bar {
fn foo() -> Wrapper<impl Sized>;
//~^ ERROR missing lifetime specifier
//~| ERROR struct takes 0 generic arguments but 1 generic argument was supplied
}

impl Bar for () {
fn foo() -> Wrapper<impl Sized> {
//~^ ERROR missing lifetime specifier
//~| ERROR struct takes 0 generic arguments but 1 generic argument was supplied
Wrapper(&())
}
}

fn main() {}
71 changes: 56 additions & 15 deletions tests/ui/impl-trait/in-trait/opaque-and-lifetime-mismatch.stderr
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
error[E0106]: missing lifetime specifier
--> $DIR/opaque-and-lifetime-mismatch.rs:5:24
--> $DIR/opaque-and-lifetime-mismatch.rs:4:24
|
LL | fn bar() -> Wrapper<impl Sized>;
| ^ expected named lifetime parameter
Expand All @@ -10,19 +10,32 @@ help: consider using the `'static` lifetime, but this is uncommon unless you're
LL | fn bar() -> Wrapper<'static, impl Sized>;
| ++++++++

error[E0412]: cannot find type `T` in this scope
--> $DIR/opaque-and-lifetime-mismatch.rs:1:22
error[E0106]: missing lifetime specifier
--> $DIR/opaque-and-lifetime-mismatch.rs:18:24
|
LL | fn foo() -> Wrapper<impl Sized>;
| ^ expected named lifetime parameter
|
= help: this function's return type contains a borrowed value, but there is no value for it to be borrowed from
help: consider using the `'static` lifetime, but this is uncommon unless you're returning a borrowed value from a `const` or a `static`, or if you will only have owned values
|
LL | fn foo() -> Wrapper<'static, impl Sized>;
| ++++++++

error[E0106]: missing lifetime specifier
--> $DIR/opaque-and-lifetime-mismatch.rs:24:24
|
LL | struct Wrapper<'rom>(T);
| ^ not found in this scope
LL | fn foo() -> Wrapper<impl Sized> {
| ^ expected named lifetime parameter
|
help: you might be missing a type parameter
= help: this function's return type contains a borrowed value, but there is no value for it to be borrowed from
help: consider using the `'static` lifetime, but this is uncommon unless you're returning a borrowed value from a `const` or a `static`, or if you will only have owned values
|
LL | struct Wrapper<'rom, T>(T);
| +++
LL | fn foo() -> Wrapper<'static, impl Sized> {
| ++++++++

error[E0107]: struct takes 0 generic arguments but 1 generic argument was supplied
--> $DIR/opaque-and-lifetime-mismatch.rs:5:17
--> $DIR/opaque-and-lifetime-mismatch.rs:4:17
|
LL | fn bar() -> Wrapper<impl Sized>;
| ^^^^^^^ ---------- help: remove this generic argument
Expand All @@ -32,11 +45,25 @@ LL | fn bar() -> Wrapper<impl Sized>;
note: struct defined here, with 0 generic parameters
--> $DIR/opaque-and-lifetime-mismatch.rs:1:8
|
LL | struct Wrapper<'rom>(T);
LL | struct Wrapper<'rom>(&'rom ());
| ^^^^^^^

error[E0107]: struct takes 0 generic arguments but 1 generic argument was supplied
--> $DIR/opaque-and-lifetime-mismatch.rs:18:17
|
LL | fn foo() -> Wrapper<impl Sized>;
| ^^^^^^^ ---------- help: remove this generic argument
| |
| expected 0 generic arguments
|
note: struct defined here, with 0 generic parameters
--> $DIR/opaque-and-lifetime-mismatch.rs:1:8
|
LL | struct Wrapper<'rom>(&'rom ());
| ^^^^^^^

error[E0053]: method `bar` has an incompatible return type for trait
--> $DIR/opaque-and-lifetime-mismatch.rs:11:17
--> $DIR/opaque-and-lifetime-mismatch.rs:10:17
|
LL | fn bar() -> i32 {
| ^^^
Expand All @@ -45,7 +72,7 @@ LL | fn bar() -> i32 {
| return type in trait

error[E0053]: method `bar` has an incompatible type for trait
--> $DIR/opaque-and-lifetime-mismatch.rs:11:17
--> $DIR/opaque-and-lifetime-mismatch.rs:10:17
|
LL | fn bar() -> i32 {
| ^^^
Expand All @@ -54,14 +81,28 @@ LL | fn bar() -> i32 {
| help: change the output type to match the trait: `Wrapper<'static>`
|
note: type in trait
--> $DIR/opaque-and-lifetime-mismatch.rs:5:17
--> $DIR/opaque-and-lifetime-mismatch.rs:4:17
|
LL | fn bar() -> Wrapper<impl Sized>;
| ^^^^^^^^^^^^^^^^^^^
= note: expected signature `fn() -> Wrapper<'static>`
found signature `fn() -> i32`

error: aborting due to 5 previous errors
error[E0107]: struct takes 0 generic arguments but 1 generic argument was supplied
--> $DIR/opaque-and-lifetime-mismatch.rs:24:17
|
LL | fn foo() -> Wrapper<impl Sized> {
| ^^^^^^^ ---------- help: remove this generic argument
| |
| expected 0 generic arguments
|
note: struct defined here, with 0 generic parameters
--> $DIR/opaque-and-lifetime-mismatch.rs:1:8
|
LL | struct Wrapper<'rom>(&'rom ());
| ^^^^^^^

error: aborting due to 8 previous errors

Some errors have detailed explanations: E0053, E0106, E0107, E0412.
Some errors have detailed explanations: E0053, E0106, E0107.
For more information about an error, try `rustc --explain E0053`.

0 comments on commit f03326c

Please sign in to comment.