diff options
| author | Owen Jacobson <owen@grimoire.ca> | 2025-08-13 14:21:10 -0400 |
|---|---|---|
| committer | Owen Jacobson <owen@grimoire.ca> | 2025-08-13 14:31:14 -0400 |
| commit | 4ace27830ffea715c30f366765aeb231572c60ec (patch) | |
| tree | d07ceb9f445bd4c9601273e1de002addcbd8c645 /src/message | |
| parent | bfcdc3fde5a39eb1d51a30c34d31330ea88b242f (diff) | |
Rust 1.89: Add elided lifetime parameters (`'_`) where appropriate.
Rust 1.89 added a new warning:
warning: hiding a lifetime that's elided elsewhere is confusing
--> src/setup/repo.rs:4:14
|
4 | fn setup(&mut self) -> Setup;
| ^^^^^^^^^ ----- the same lifetime is hidden here
| |
| the lifetime is elided here
|
= help: the same lifetime is referred to in inconsistent ways, making the signature confusing
help: use `'_` for type paths
|
4 | fn setup(&mut self) -> Setup<'_>;
| ++++
I don't entirely agree with the style advice here, but lifetime elision style is an evolving area in Rust and I'd rather track the Rust team's recommendations than invent my own, so I've added all of them.
Diffstat (limited to 'src/message')
| -rw-r--r-- | src/message/repo.rs | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/message/repo.rs b/src/message/repo.rs index 9b65a67..b4c086d 100644 --- a/src/message/repo.rs +++ b/src/message/repo.rs @@ -9,11 +9,11 @@ use crate::{ }; pub trait Provider { - fn messages(&mut self) -> Messages; + fn messages(&mut self) -> Messages<'_>; } impl Provider for Transaction<'_, Sqlite> { - fn messages(&mut self) -> Messages { + fn messages(&mut self) -> Messages<'_> { Messages(self) } } |
