From b3f4a9015c9ee118574bb9d3419b874826552c24 Mon Sep 17 00:00:00 2001 From: Kevin Reid Date: Sat, 18 Nov 2023 10:53:03 -0800 Subject: [PATCH] Remove space and rephrase `map()` advice. --- clippy_lints/src/unnecessary_map_on_constructor.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/clippy_lints/src/unnecessary_map_on_constructor.rs b/clippy_lints/src/unnecessary_map_on_constructor.rs index 39a83bfa005..5c880288430 100644 --- a/clippy_lints/src/unnecessary_map_on_constructor.rs +++ b/clippy_lints/src/unnecessary_map_on_constructor.rs @@ -10,11 +10,11 @@ use rustc_span::sym; declare_clippy_lint! { /// ### What it does /// Suggests removing the use of a `map()` (or `map_err()`) method when an `Option` or `Result` - /// is being constructed. + /// is being constructed. /// /// ### Why is this bad? - /// It introduces unnecessary complexity. In this case the function can be used directly and - /// construct the `Option` or `Result` from the output. + /// It introduces unnecessary complexity. Instead, the function can be called before + /// constructing the `Option` or `Result` from its return value. /// /// ### Example /// ```no_run