about summary refs log tree commit diff
diff options
context:
space:
mode:
authorKevin Reid <kpreid@switchb.org>2023-11-18 10:53:03 -0800
committerKevin Reid <kpreid@switchb.org>2023-11-18 10:53:03 -0800
commitb3f4a9015c9ee118574bb9d3419b874826552c24 (patch)
treeab1c675024f5550f95330057dded67a56548c667
parentbffb664fbf094f53d51bec66d1b042265db1cbad (diff)
Remove space and rephrase `map()` advice.
-rw-r--r--clippy_lints/src/unnecessary_map_on_constructor.rs6
1 files 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