Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Commit

Permalink
BlockId removal: refactor: Finalizer
Browse files Browse the repository at this point in the history
It changes the arguments of methods of `Finalizer` trait from: block:
`BlockId<Block>` to: hash: `&Block::Hash`

This PR is part of BlockId::Number refactoring analysis (paritytech/substrate#11292)
  • Loading branch information
michalkucharczyk committed Oct 19, 2022
1 parent 7273afa commit 2b90639
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion client/consensus/common/src/parachain_consensus.rs
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ where

// don't finalize the same block multiple times.
if parachain.usage_info().chain.finalized_hash != hash {
if let Err(e) = parachain.finalize_block(BlockId::hash(hash), None, true) {
if let Err(e) = parachain.finalize_block(&hash, None, true) {
match e {
ClientError::UnknownBlock(_) => tracing::debug!(
target: "cumulus-consensus",
Expand Down

0 comments on commit 2b90639

Please sign in to comment.