refactor: prefer if let to single variant match

This commit is contained in:
Lucille L. Blumire 2025-04-17 16:05:47 +01:00
parent 2ff169da9f
commit 36afc85d38
No known key found for this signature in database
GPG key ID: D168492023622329

View file

@ -292,19 +292,16 @@ fn parse_batch_range(s: &str) -> error::Result<(L1BatchNumber, L1BatchNumber)> {
.map(L1BatchNumber::from) .map(L1BatchNumber::from)
.map_err(|e| error::Error::internal(format!("Can't convert batch {s} to number: {e}"))) .map_err(|e| error::Error::internal(format!("Can't convert batch {s} to number: {e}")))
}; };
match s.split_once('-') { if let Some((start, end)) = s.split_once('-') {
Some((start, end)) => { let (start, end) = (parse(start)?, parse(end)?);
let (start, end) = (parse(start)?, parse(end)?); if start > end {
if start > end { Err(error::Error::InvalidBatchRange(s.into()))
Err(error::Error::InvalidBatchRange(s.into())) } else {
} else { Ok((start, end))
Ok((start, end))
}
}
None => {
let batch_number = parse(s)?;
Ok((batch_number, batch_number))
} }
} else {
let batch_number = parse(s)?;
Ok((batch_number, batch_number))
} }
} }