HomePhabricator

[avalanche] Simplify the proof verification code in the peer manager

Description

[avalanche] Simplify the proof verification code in the peer manager

Summary:
As per review https://reviews.bitcoinabc.org/D10522#inline-49160, there is no point passing the state as a parameter if it is already captured. This makes it possible to simplify the code a bit.

Ref T1854.

Test Plan:

ninja all check-avalanche

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Maniphest Tasks: T1854

Differential Revision: https://reviews.bitcoinabc.org/D10556

Details

Provenance
FabienAuthored on Nov 26 2021, 09:50
FabienPushed on Nov 26 2021, 15:18
Reviewer
Restricted Project
Differential Revision
D10556: [avalanche] Simplify the proof verification code in the peer manager
Parents
rABCeeb56267d9d4: [avalanche] Don't create invalid proofs in the conflicting_proof_selection test
Branches
Unknown
Tags
Unknown
Tasks
Restricted Maniphest Task