@@ -696,7 +696,7 @@ impl<N: Network> Service<N> {
696696 . await ;
697697
698698 let mut batch_start = stream_start;
699- let mut batch_end = match provider. get_block_by_number ( confirmed. into ( ) ) . await {
699+ let mut previous_batch_end = match provider. get_block_by_number ( confirmed. into ( ) ) . await {
700700 Ok ( block) => Some ( block) ,
701701 Err ( e) => {
702702 error ! ( batch_start = batch_start, batch_end = confirmed, error = %e, "Failed to get initial batch end block" ) ;
@@ -709,7 +709,7 @@ impl<N: Network> Service<N> {
709709 let incoming_block_num = incoming_block. number ( ) ;
710710 info ! ( block_number = incoming_block_num, "Received block header" ) ;
711711
712- let reorged_opt = match batch_end . as_ref ( ) {
712+ let reorged_opt = match previous_batch_end . as_ref ( ) {
713713 None => None ,
714714 Some ( batch_end) => match reorg_handler. check ( batch_end) . await {
715715 Ok ( opt) => opt,
@@ -728,10 +728,10 @@ impl<N: Network> Service<N> {
728728 // no need to stream blocks prior to the previously specified starting block
729729 if common_ancestor. header ( ) . number ( ) < stream_start {
730730 batch_start = stream_start;
731- batch_end = None ;
731+ previous_batch_end = None ;
732732 } else {
733733 batch_start = common_ancestor. header ( ) . number ( ) + 1 ;
734- batch_end = Some ( common_ancestor) ;
734+ previous_batch_end = Some ( common_ancestor) ;
735735 }
736736
737737 // TODO: explain in docs that the returned block after a reorg will be the
@@ -742,8 +742,8 @@ impl<N: Network> Service<N> {
742742 // no reorg happened, move the block range back to expected next start
743743 //
744744 // SAFETY: Overflow cannot realistically happen
745- if let Some ( batch_end ) = batch_end . as_ref ( ) {
746- batch_start = batch_end . header ( ) . number ( ) + 1 ;
745+ if let Some ( prev_batch_end ) = previous_batch_end . as_ref ( ) {
746+ batch_start = prev_batch_end . header ( ) . number ( ) + 1 ;
747747 }
748748 }
749749
@@ -754,7 +754,10 @@ impl<N: Network> Service<N> {
754754 // reads
755755 let batch_end_num =
756756 confirmed. min ( batch_start. saturating_add ( max_block_range - 1 ) ) ;
757- batch_end = match provider. get_block_by_number ( batch_end_num. into ( ) ) . await {
757+ previous_batch_end = match provider
758+ . get_block_by_number ( batch_end_num. into ( ) )
759+ . await
760+ {
758761 Ok ( block) => Some ( block) ,
759762 Err ( e) => {
760763 error ! ( batch_start = batch_start, batch_end = batch_end_num, error = %e, "Failed to get ending block of the current batch" ) ;
0 commit comments