use let-else to reduce nesting
..and create a nice linear structure
This commit is contained in:
parent
066bdb2155
commit
22bbb4eba2
1 changed files with 8 additions and 8 deletions
|
|
@ -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)
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue