1775e7b93d
Do not suggest constraining the `&self` param, but rather the return type. If that is wrong (because it is not sufficient), a follow up error will tell the user to fix it. This way we lower the chances of *over* constraining, but still get the cake of "correctly" contrained in two steps. This is a correct suggestion: ``` error: lifetime may not live long enough --> $DIR/ex3-both-anon-regions-return-type-is-anon.rs:9:9 | LL | fn foo<'a>(&self, x: &i32) -> &i32 { | - - let's call the lifetime of this reference `'1` | | | let's call the lifetime of this reference `'2` LL | x | ^ method was supposed to return data with lifetime `'2` but it is returning data with lifetime `'1` | help: consider introducing a named lifetime parameter and update trait if needed | LL | fn foo<'a>(&self, x: &'a i32) -> &'a i32 { | ++ ++ ``` While this is incomplete because it should suggestino `&'a self` ``` error: lifetime may not live long enough --> $DIR/ex3-both-anon-regions-self-is-anon.rs:7:19 | LL | fn foo<'a>(&self, x: &Foo) -> &Foo { | - - let's call the lifetime of this reference `'1` | | | let's call the lifetime of this reference `'2` LL | if true { x } else { self } | ^ method was supposed to return data with lifetime `'2` but it is returning data with lifetime `'1` | help: consider introducing a named lifetime parameter and update trait if needed | LL | fn foo<'a>(&self, x: &'a Foo) -> &'a Foo { | ++ ++ ``` but the follow up error is ``` error: lifetime may not live long enough --> tests/ui/lifetimes/lifetime-errors/ex3-both-anon-regions-self-is-anon.rs:7:30 | 6 | fn foo<'a>(&self, x: &'a Foo) -> &'a Foo { | -- - let's call the lifetime of this reference `'1` | | | lifetime `'a` defined here 7 | if true { x } else { self } | ^^^^ method was supposed to return data with lifetime `'a` but it is returning data with lifetime `'1` | help: consider introducing a named lifetime parameter and update trait if needed | 6 | fn foo<'a>(&'a self, x: &'a Foo) -> &'a Foo { | ++ ```
47 lines
1.0 KiB
Rust
47 lines
1.0 KiB
Rust
//@ run-rustfix
|
|
#![allow(non_snake_case, dead_code)]
|
|
|
|
use std::pin::Pin;
|
|
|
|
struct Struct<'a> {
|
|
data: &'a u32,
|
|
}
|
|
|
|
impl<'a> Struct<'a> {
|
|
// Test using `&self` sugar:
|
|
|
|
fn ref_self<'b>(&self, f: &'b u32) -> &'b u32 {
|
|
f
|
|
//~^ ERROR lifetime may not live long enough
|
|
}
|
|
|
|
// Test using `&Self` explicitly:
|
|
|
|
fn ref_Self<'b>(self: &Self, f: &'b u32) -> &'b u32 {
|
|
f
|
|
//~^ ERROR lifetime may not live long enough
|
|
}
|
|
|
|
fn box_ref_Self<'b>(self: Box<&Self>, f: &'b u32) -> &'b u32 {
|
|
f
|
|
//~^ ERROR lifetime may not live long enough
|
|
}
|
|
|
|
fn pin_ref_Self<'b>(self: Pin<&Self>, f: &'b u32) -> &'b u32 {
|
|
f
|
|
//~^ ERROR lifetime may not live long enough
|
|
}
|
|
|
|
fn box_box_ref_Self<'b>(self: Box<Box<&Self>>, f: &'b u32) -> &'b u32 {
|
|
f
|
|
//~^ ERROR lifetime may not live long enough
|
|
}
|
|
|
|
fn box_pin_Self<'b>(self: Box<Pin<&Self>>, f: &'b u32) -> &'b u32 {
|
|
f
|
|
//~^ ERROR lifetime may not live long enough
|
|
}
|
|
}
|
|
|
|
fn main() {}
|