From 36afc85d3891fac8aa36ce1ecfd81b6dc11d182b Mon Sep 17 00:00:00 2001 From: "Lucille L. Blumire" Date: Thu, 17 Apr 2025 16:05:47 +0100 Subject: [PATCH] refactor: prefer if let to single variant match --- .../src/core/config.rs | 21 ++++++++----------- 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/bin/verify-era-proof-attestation/src/core/config.rs b/bin/verify-era-proof-attestation/src/core/config.rs index 4e0607f..c6f219b 100644 --- a/bin/verify-era-proof-attestation/src/core/config.rs +++ b/bin/verify-era-proof-attestation/src/core/config.rs @@ -292,19 +292,16 @@ fn parse_batch_range(s: &str) -> error::Result<(L1BatchNumber, L1BatchNumber)> { .map(L1BatchNumber::from) .map_err(|e| error::Error::internal(format!("Can't convert batch {s} to number: {e}"))) }; - match s.split_once('-') { - Some((start, end)) => { - let (start, end) = (parse(start)?, parse(end)?); - if start > end { - Err(error::Error::InvalidBatchRange(s.into())) - } else { - Ok((start, end)) - } - } - None => { - let batch_number = parse(s)?; - Ok((batch_number, batch_number)) + if let Some((start, end)) = s.split_once('-') { + let (start, end) = (parse(start)?, parse(end)?); + if start > end { + Err(error::Error::InvalidBatchRange(s.into())) + } else { + Ok((start, end)) } + } else { + let batch_number = parse(s)?; + Ok((batch_number, batch_number)) } }