Skip to content

Comments

chore: remove redundant if statement#6387

Closed
cuiweixie wants to merge 1 commit intographprotocol:masterfrom
cuiweixie:bugfix
Closed

chore: remove redundant if statement#6387
cuiweixie wants to merge 1 commit intographprotocol:masterfrom
cuiweixie:bugfix

Conversation

@cuiweixie
Copy link

@cuiweixie cuiweixie commented Feb 22, 2026

these two statements is duplicated.

if to > head_ptr.number - reorg_threshold {
return Ok(ReconciliationStep::Retry);
}
if to > head_ptr.number - reorg_threshold {
return Ok(ReconciliationStep::Retry);
}

@lutter
Copy link
Collaborator

lutter commented Feb 23, 2026

Nice catch! Thanks for the contribution.

Since the commit wasn't signed, I couldn't merge the PR. But I committed an identical change as 68fc7f6

@lutter lutter closed this Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants