diff options
| author | Samuel Tardieu <sam@rfc1149.net> | 2025-02-02 19:11:14 +0100 |
|---|---|---|
| committer | Samuel Tardieu <sam@rfc1149.net> | 2025-02-15 15:56:41 +0100 |
| commit | 510d3b69fc2e13f7cc2cb485716aba6908f147dc (patch) | |
| tree | 09b309d2e0de8b19599050616ad2b918fcfd03db | |
| parent | e0a62acc8826875dcf2ff17750f6c28fcc69c2f4 (diff) | |
| download | rust-510d3b69fc2e13f7cc2cb485716aba6908f147dc.tar.gz rust-510d3b69fc2e13f7cc2cb485716aba6908f147dc.zip | |
Use `clippy_utils::is_mutable()` in `unnecessary_struct_initialization`
| -rw-r--r-- | clippy_lints/src/unnecessary_struct_initialization.rs | 14 |
1 files changed, 2 insertions, 12 deletions
diff --git a/clippy_lints/src/unnecessary_struct_initialization.rs b/clippy_lints/src/unnecessary_struct_initialization.rs index 1df229c330e..5792b6b3178 100644 --- a/clippy_lints/src/unnecessary_struct_initialization.rs +++ b/clippy_lints/src/unnecessary_struct_initialization.rs @@ -1,8 +1,8 @@ use clippy_utils::diagnostics::span_lint_and_sugg; use clippy_utils::source::snippet; use clippy_utils::ty::is_copy; -use clippy_utils::{get_parent_expr, path_to_local}; -use rustc_hir::{BindingMode, Expr, ExprField, ExprKind, Node, PatKind, Path, QPath, StructTailExpr, UnOp}; +use clippy_utils::{get_parent_expr, is_mutable, path_to_local}; +use rustc_hir::{Expr, ExprField, ExprKind, Path, QPath, StructTailExpr, UnOp}; use rustc_lint::{LateContext, LateLintPass}; use rustc_session::declare_lint_pass; @@ -157,16 +157,6 @@ fn same_path_in_all_fields<'tcx>( } } -fn is_mutable(cx: &LateContext<'_>, expr: &Expr<'_>) -> bool { - if let Some(hir_id) = path_to_local(expr) - && let Node::Pat(pat) = cx.tcx.hir_node(hir_id) - { - matches!(pat.kind, PatKind::Binding(BindingMode::MUT, ..)) - } else { - true - } -} - fn check_references(cx: &LateContext<'_>, expr_a: &Expr<'_>, expr_b: &Expr<'_>) -> bool { if let Some(parent) = get_parent_expr(cx, expr_a) && let parent_ty = cx.typeck_results().expr_ty_adjusted(parent) |
