Skip to content

Commit 1cc4d0f

Browse files
committed
inline suggestion and use path.ident.name instead of snippet
1 parent d3cba50 commit 1cc4d0f

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

clippy_lints/src/unnecessary_map_on_constructor.rs

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -75,19 +75,17 @@ impl<'tcx> LateLintPass<'tcx> for UnnecessaryMapOnConstructor {
7575
if let hir::ExprKind::Path(fun) = arg.kind;
7676
then {
7777
let mut applicability = Applicability::MachineApplicable;
78-
let fun_snippet = snippet_with_applicability(cx, fun.span(), "_", &mut applicability);
7978
let constructor_snippet =
8079
snippet_with_applicability(cx, constructor_path.span(), "_", &mut applicability);
8180
let constructor_arg_snippet =
8281
snippet_with_applicability(cx, constructor_item.span, "_", &mut applicability);
83-
let suggestion = format!("{constructor_snippet}({fun_snippet}({constructor_arg_snippet}))");
8482
span_lint_and_sugg(
8583
cx,
8684
UNNECESSARY_MAP_ON_CONSTRUCTOR,
8785
expr.span,
88-
&format!("unnecessary {fun_snippet} on contstuctor {constructor_snippet}(_)"),
89-
"try using",
90-
suggestion,
86+
&format!("unnecessary {} on contstuctor {constructor_snippet}(_)", path.ident.name),
87+
"try",
88+
format!("{constructor_snippet}({}({constructor_arg_snippet}))", path.ident.name),
9189
applicability,
9290
);
9391
}

0 commit comments

Comments
 (0)