Page MenuHomePhabricator

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

Authored by Fabien on Nov 26 2021, 09:53.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Maniphest Tasks
Restricted Maniphest Task
Commits
rABC1612135f568a: [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

Diff Detail

Repository
rABC Bitcoin ABC
Branch
avalanche_simplify_valid_lambda
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 17388
Build 34604: Build Diffbuild-without-wallet · build-clang-tidy · lint-circular-dependencies · build-diff · build-clang · build-debug
Build 34603: arc lint + arc unit