about summary refs log tree commit diff
diff options
context:
space:
mode:
authordarklyspaced <srohanjd@gmail.com>2023-06-27 11:28:45 +0800
committerdarklyspaced <srohanjd@gmail.com>2023-06-27 11:29:02 +0800
commita693cbc1bca16c753dfb1aa853e970bd2028634c (patch)
tree445e22117fb12fda69594c81f890d63123031e9e
parentca00b138b21d87cff24d085dd9d9dce2bf26b530 (diff)
downloadrust-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.rs13
-rw-r--r--clippy_utils/src/msrvs.rs2
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 }