diff options
| author | dswij <dswijj@gmail.com> | 2022-01-25 16:18:32 +0800 |
|---|---|---|
| committer | dswij <dswijj@gmail.com> | 2022-01-28 12:00:30 +0800 |
| commit | 515ed80b9d63e20c196a7739118c2f316a097106 (patch) | |
| tree | 0f4bc06618f83127539c05df0314e6dc034d4cde | |
| parent | ea4db3a6c6fdd30abb0b746f7a68d403a07d24bf (diff) | |
| download | rust-515ed80b9d63e20c196a7739118c2f316a097106.tar.gz rust-515ed80b9d63e20c196a7739118c2f316a097106.zip | |
Update docs for `map_flatten` on `Option`
| -rw-r--r-- | clippy_lints/src/methods/mod.rs | 5 |
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, |
