diff options
| author | darklyspaced <srohanjd@gmail.com> | 2023-06-27 11:28:45 +0800 |
|---|---|---|
| committer | darklyspaced <srohanjd@gmail.com> | 2023-06-27 11:29:02 +0800 |
| commit | a693cbc1bca16c753dfb1aa853e970bd2028634c (patch) | |
| tree | 445e22117fb12fda69594c81f890d63123031e9e | |
| parent | ca00b138b21d87cff24d085dd9d9dce2bf26b530 (diff) | |
| download | rust-a693cbc1bca16c753dfb1aa853e970bd2028634c.tar.gz rust-a693cbc1bca16c753dfb1aa853e970bd2028634c.zip | |
changed msrv name for consistency & changed nested if for style
| -rw-r--r-- | clippy_lints/src/methods/option_map_unwrap_or.rs | 13 | ||||
| -rw-r--r-- | clippy_utils/src/msrvs.rs | 2 |
2 files changed, 5 insertions, 10 deletions
diff --git a/clippy_lints/src/methods/option_map_unwrap_or.rs b/clippy_lints/src/methods/option_map_unwrap_or.rs index e836a1395a2..f4f158c0439 100644 --- a/clippy_lints/src/methods/option_map_unwrap_or.rs +++ b/clippy_lints/src/methods/option_map_unwrap_or.rs @@ -77,15 +77,10 @@ pub(super) fn check<'tcx>( return; } - let mut suggest_is_some_and = false; - // argument to `unwrap_or` is false & is_some_and is stabilised; should suggest using `is_some_and` - if msrv.meets(msrvs::OPT_IS_SOME_AND) { - if let ExprKind::Lit(unwrap_lit) = &unwrap_arg.kind { - if let rustc_ast::LitKind::Bool(false) = unwrap_lit.node { - suggest_is_some_and = true; - } - } - } + // is_some_and is stabilised && `unwrap_or` argument is false; suggest `is_some_and` instead + let suggest_is_some_and = msrv.meets(msrvs::OPTION_IS_SOME_AND) + && matches!(&unwrap_arg.kind, ExprKind::Lit(lit) + if matches!(lit.node, rustc_ast::LitKind::Bool(false))); let mut applicability = Applicability::MachineApplicable; // get snippet for unwrap_or() diff --git a/clippy_utils/src/msrvs.rs b/clippy_utils/src/msrvs.rs index 1541dc8392c..5aacca45146 100644 --- a/clippy_utils/src/msrvs.rs +++ b/clippy_utils/src/msrvs.rs @@ -19,7 +19,7 @@ macro_rules! msrv_aliases { // names may refer to stabilized feature flags or library items msrv_aliases! { - 1,70,0 { OPT_IS_SOME_AND } + 1,70,0 { OPTION_IS_SOME_AND } 1,68,0 { PATH_MAIN_SEPARATOR_STR } 1,65,0 { LET_ELSE, POINTER_CAST_CONSTNESS } 1,62,0 { BOOL_THEN_SOME, DEFAULT_ENUM_ATTRIBUTE } |
