diff options
| author | Philipp Krones <hello@philkrones.com> | 2024-01-26 09:42:12 +0100 |
|---|---|---|
| committer | Philipp Krones <hello@philkrones.com> | 2024-01-26 09:42:12 +0100 |
| commit | de8ccdbf80e7014092341096663a9dbba663bebb (patch) | |
| tree | 671be4519a611c52ac69a748d3c2d57dcab81c40 | |
| parent | d7a0182157de02a49f1a3e219bcd8c29b388be41 (diff) | |
| download | rust-de8ccdbf80e7014092341096663a9dbba663bebb.tar.gz rust-de8ccdbf80e7014092341096663a9dbba663bebb.zip | |
Clippy: Fix empty suggestion in from_over_into
Co-authored-by: y21 <30553356+y21@users.noreply.github.com>
| -rw-r--r-- | src/tools/clippy/clippy_lints/src/from_over_into.rs | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/tools/clippy/clippy_lints/src/from_over_into.rs b/src/tools/clippy/clippy_lints/src/from_over_into.rs index 93527bcdf5c..1933a00891b 100644 --- a/src/tools/clippy/clippy_lints/src/from_over_into.rs +++ b/src/tools/clippy/clippy_lints/src/from_over_into.rs @@ -181,9 +181,6 @@ fn convert_to_from( let from = snippet_opt(cx, self_ty.span)?; let into = snippet_opt(cx, target_ty.span)?; - let return_type = matches!(sig.decl.output, FnRetTy::Return(_)) - .then_some(String::from("Self")) - .unwrap_or_default(); let mut suggestions = vec![ // impl Into<T> for U -> impl From<T> for U // ~~~~ ~~~~ @@ -200,10 +197,13 @@ fn convert_to_from( // fn into([mut] self) -> T -> fn into([mut] v: T) -> T // ~~~~ ~~~~ (self_ident.span, format!("val: {from}")), + ]; + + if let FnRetTy::Return(_) = sig.decl.output { // fn into(self) -> T -> fn into(self) -> Self // ~ ~~~~ - (sig.decl.output.span(), return_type), - ]; + suggestions.push((sig.decl.output.span(), String::from("Self"))); + } let mut finder = SelfFinder { cx, |
