Skip to content

Conversation

@wen-coding
Copy link
Contributor

Currently highest_finalized_slot is just used to:

  1. check whether the returned highest_finalized_slot is larger than the local copy, which certificate_pool already checks before returning
  2. used in Standstill event creation, here we can actually get the value from cert_pool.highest_finalized_slot().

So we can remove the local highest_finalized_slot, makes the code a bit simpler.

@wen-coding wen-coding merged commit e3944ae into anza-xyz:master Jul 28, 2025
7 checks passed
@wen-coding wen-coding deleted the remove_local_new_finalized_slot branch July 28, 2025 23:34
wen-coding added a commit to wen-coding/alpenglow that referenced this pull request Jul 29, 2025
bw-solana pushed a commit to bw-solana/alpenglow that referenced this pull request Aug 1, 2025
bw-solana pushed a commit to bw-solana/alpenglow that referenced this pull request Aug 1, 2025
bw-solana pushed a commit to bw-solana/alpenglow that referenced this pull request Aug 1, 2025
bw-solana pushed a commit to bw-solana/alpenglow that referenced this pull request Aug 1, 2025
bw-solana pushed a commit to bw-solana/alpenglow that referenced this pull request Aug 1, 2025
bw-solana pushed a commit to bw-solana/alpenglow that referenced this pull request Aug 1, 2025
bw-solana pushed a commit to bw-solana/alpenglow that referenced this pull request Aug 2, 2025
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