diff options
| author | Michael Goulet <michael@errs.io> | 2023-12-29 05:16:53 +0000 |
|---|---|---|
| committer | Michael Goulet <michael@errs.io> | 2023-12-29 05:16:53 +0000 |
| commit | 3fceca23bb64e304df56b3bd86d26790b5301bdf (patch) | |
| tree | db71be289c874764c2f3cb85d1f3b539654fbac7 | |
| parent | ac4c2094a6030530661bee3876e0228ddfeb6b8b (diff) | |
| download | rust-3fceca23bb64e304df56b3bd86d26790b5301bdf.tar.gz rust-3fceca23bb64e304df56b3bd86d26790b5301bdf.zip | |
Remove mitigations for incorrect node args
| -rw-r--r-- | clippy_lints/src/methods/unnecessary_to_owned.rs | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/clippy_lints/src/methods/unnecessary_to_owned.rs b/clippy_lints/src/methods/unnecessary_to_owned.rs index 637368e9361..5f69cf6cd7a 100644 --- a/clippy_lints/src/methods/unnecessary_to_owned.rs +++ b/clippy_lints/src/methods/unnecessary_to_owned.rs @@ -445,11 +445,12 @@ fn can_change_type<'a>(cx: &LateContext<'a>, mut expr: &'a Expr<'a>, mut ty: Ty< { let bound_fn_sig = cx.tcx.fn_sig(callee_def_id); let fn_sig = bound_fn_sig.skip_binder(); - if let Some(arg_index) = recv.into_iter().chain(call_args).position(|arg| arg.hir_id == expr.hir_id) + if let Some(arg_index) = recv + .into_iter() + .chain(call_args) + .position(|arg| arg.hir_id == expr.hir_id) && let param_ty = fn_sig.input(arg_index).skip_binder() - && let ty::Param(ParamTy { index: param_index , ..}) = *param_ty.kind() - // https://github.com/rust-lang/rust-clippy/issues/9504 and https://github.com/rust-lang/rust-clippy/issues/10021 - && (param_index as usize) < call_generic_args.len() + && let ty::Param(ParamTy { index: param_index, .. }) = *param_ty.kind() { if fn_sig .skip_binder() |