diff options
| author | Ada Alakbarova <ada.alakbarova@proton.me> | 2025-08-14 23:17:36 +0200 |
|---|---|---|
| committer | Ada Alakbarova <ada.alakbarova@proton.me> | 2025-08-14 23:17:54 +0200 |
| commit | 22bbb4eba2c0b84dd7adda00280b2ae8d929a3eb (patch) | |
| tree | 17ab9ecf4ffba6d4c322501f5cdd6f1b684cd799 | |
| parent | 066bdb2155b21a9e36e0f59b64ac3df91b6b10b4 (diff) | |
use `let-else` to reduce nesting
..and create a nice linear structure
| -rw-r--r-- | clippy_utils/src/msrvs.rs | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/clippy_utils/src/msrvs.rs b/clippy_utils/src/msrvs.rs index 3c96b15a99d..896d607fbcd 100644 --- a/clippy_utils/src/msrvs.rs +++ b/clippy_utils/src/msrvs.rs @@ -198,16 +198,16 @@ fn parse_attrs(sess: &Session, attrs: &[impl AttributeExt]) -> Option<RustcVersi .emit(); } - if let Some(msrv) = msrv_attr.value_str() { - if let Some(version) = parse_version(msrv) { - return Some(version); - } + let Some(msrv) = msrv_attr.value_str() else { + sess.dcx().span_err(msrv_attr.span(), "bad clippy attribute"); + return None; + }; + let Some(version) = parse_version(msrv) else { sess.dcx() .span_err(msrv_attr.span(), format!("`{msrv}` is not a valid Rust version")); - } else { - sess.dcx().span_err(msrv_attr.span(), "bad clippy attribute"); - } + return None; + }; - None + Some(version) } |
