about summary refs log tree commit diff
diff options
context:
space:
mode:
authordswij <dswijj@gmail.com>2022-01-25 16:18:32 +0800
committerdswij <dswijj@gmail.com>2022-01-28 12:00:30 +0800
commit515ed80b9d63e20c196a7739118c2f316a097106 (patch)
tree0f4bc06618f83127539c05df0314e6dc034d4cde
parentea4db3a6c6fdd30abb0b746f7a68d403a07d24bf (diff)
downloadrust-515ed80b9d63e20c196a7739118c2f316a097106.tar.gz
rust-515ed80b9d63e20c196a7739118c2f316a097106.zip
Update docs for `map_flatten` on `Option`
-rw-r--r--clippy_lints/src/methods/mod.rs5
1 files changed, 4 insertions, 1 deletions
diff --git a/clippy_lints/src/methods/mod.rs b/clippy_lints/src/methods/mod.rs
index d75ab0c4b1b..45f2992df91 100644
--- a/clippy_lints/src/methods/mod.rs
+++ b/clippy_lints/src/methods/mod.rs
@@ -566,17 +566,20 @@ declare_clippy_lint! {
     ///
     /// ### Why is this bad?
     /// Readability, this can be written more concisely as
-    /// `_.flat_map(_)`
+    /// `_.flat_map(_)` for `Iterator` or `_.and_then(_)` for `Option`
     ///
     /// ### Example
     /// ```rust
     /// let vec = vec![vec![1]];
+    /// let opt = Some(5);
     ///
     /// // Bad
     /// vec.iter().map(|x| x.iter()).flatten();
+    /// opt.map(|x| Some(x * 2)).flatten();
     ///
     /// // Good
     /// vec.iter().flat_map(|x| x.iter());
+    /// opt.and_then(|x| Some(x * 2));
     /// ```
     #[clippy::version = "1.31.0"]
     pub MAP_FLATTEN,