chore: simplify Consensus::validate_header_range with let-else (#5659)

This commit is contained in:
DaniPopes
2023-12-01 23:30:43 +01:00
committed by GitHub
parent a53e2c0666
commit 5ac4a3d4cb

View File

@ -37,18 +37,16 @@ pub trait Consensus: Debug + Send + Sync {
///
/// Note: this expects that the headers are in natural order (ascending block number)
fn validate_header_range(&self, headers: &[SealedHeader]) -> Result<(), ConsensusError> {
if headers.is_empty() {
return Ok(())
}
let first = headers.first().expect("checked empty");
self.validate_header(first)?;
let mut parent = first;
for child in headers.iter().skip(1) {
let mut headers = headers.iter();
let Some(mut parent) = headers.next() else {
return Ok(());
};
self.validate_header(parent)?;
for child in headers {
self.validate_header(child)?;
self.validate_header_against_parent(child, parent)?;
parent = child;
}
Ok(())
}