Mirrors/rust-clippy
2
0
Fork 0
mirror of https://github.com/rust-lang/rust-clippy synced 2024-11-15 01:17:16 +00:00
Code Issues Projects Releases Packages Wiki Activity

Use more concrete explanation for methods

Browse source 
Show some code rather than "a single method call".
This commit is contained in:
Ivan Tham 2020-10-06 23:58:32 +08:00 committed by GitHub
parent c9fdeef643
commit 7021d70d2e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
clippy_lints/src/methods/mod.rs
View file

@ -400,8 +400,8 @@ declare_clippy_lint! {
declare_clippy_lint! {
/// **What it does:** Checks for usage of `_.map(_).flatten(_)`,
///
/// **Why is this bad?** Readability, this can be written more concisely as a
/// single method call using `_.flat_map(_)`
/// **Why is this bad?** Readability, this can be written more concisely as
/// `_.flat_map(_)`
///
/// **Known problems:**
///
@ -424,8 +424,8 @@ declare_clippy_lint! {
/// **What it does:** Checks for usage of `_.filter(_).map(_)`,
/// `_.filter(_).flat_map(_)`, `_.filter_map(_).flat_map(_)` and similar.
///
/// **Why is this bad?** Readability, this can be written more concisely as a
/// single method call.
/// **Why is this bad?** Readability, this can be written more concisely as
/// `_.filter_map(_)`.
///
/// **Known problems:** Often requires a condition + Option/Iterator creation
/// inside the closure.
@ -452,8 +452,8 @@ declare_clippy_lint! {
declare_clippy_lint! {
/// **What it does:** Checks for usage of `_.filter_map(_).next()`.
///
/// **Why is this bad?** Readability, this can be written more concisely as a
/// single method call.
/// **Why is this bad?** Readability, this can be written more concisely as
/// `_.find_map(_)`.
///
/// **Known problems:** None
///
@ -496,8 +496,8 @@ declare_clippy_lint! {
declare_clippy_lint! {
/// **What it does:** Checks for usage of `_.find(_).map(_)`.
///
/// **Why is this bad?** Readability, this can be written more concisely as a
/// single method call.
/// **Why is this bad?** Readability, this can be written more concisely as
/// `_.find_map(_)`.
///
/// **Known problems:** Often requires a condition + Option/Iterator creation
/// inside the closure.
@ -1276,8 +1276,8 @@ declare_clippy_lint! {
declare_clippy_lint! {
/// **What it does:** Checks for usage of `_.as_ref().map(Deref::deref)` or it's aliases (such as String::as_str).
///
/// **Why is this bad?** Readability, this can be written more concisely as a
/// single method call.
/// **Why is this bad?** Readability, this can be written more concisely as
/// `_.as_deref()`.
///
/// **Known problems:** None.
///