grassrootseconomics/openethereum
5
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

fix reverse start block check

Browse Source
This commit is contained in:
Robert Habermeier 2016-12-16 23:21:59 +01:00
parent 08587a1f56
commit e57ab96731
1 changed files with 5 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
sync/src/light_sync/response.rs
View File

@ -96,10 +96,8 @@ struct Max(usize);
impl Constraint for StartsAtNumber { impl Constraint for StartsAtNumber {
type Error = BasicError; type Error = BasicError;
fn verify(&self, headers: &[Header], reverse: bool) -> Result<(), BasicError> { fn verify(&self, headers: &[Header], _reverse: bool) -> Result<(), BasicError> {
let earliest = if reverse { headers.last() } else { headers.first() }; headers.first().map_or(Ok(()), |h| {
earliest.map_or(Ok(()), |h| {
if h.number() == self.0 { if h.number() == self.0 {
Ok(()) Ok(())
} else { } else {
@ -112,10 +110,8 @@ impl Constraint for StartsAtNumber {
impl Constraint for StartsAtHash { impl Constraint for StartsAtHash {
type Error = BasicError; type Error = BasicError;
fn verify(&self, headers: &[Header], reverse: bool) -> Result<(), BasicError> { fn verify(&self, headers: &[Header], _reverse: bool) -> Result<(), BasicError> {
let earliest = if reverse { headers.last() } else { headers.first() }; headers.first().map_or(Ok(()), |h| {
earliest.map_or(Ok(()), |h| {
if h.hash() == self.0 { if h.hash() == self.0 {
Ok(()) Ok(())
} else { } else {
@ -188,7 +184,7 @@ mod tests {
#[test] #[test]
fn sequential_backward() { fn sequential_backward() {
let request = HeadersRequest { let request = HeadersRequest {
start: 10.into(), start: 34.into(),
max: 30, max: 30,
skip: 0, skip: 0,
reverse: true, reverse: true,