Page MenuHomePhabricator

[avalanche] Use the proof registration state in net processing
ClosedPublic

Authored by Fabien on Jan 6 2022, 13:53.

Details

Reviewers
tyler-smith
Group Reviewers
Restricted Project
Maniphest Tasks
Restricted Maniphest Task
Commits
rABC216ecee51737: [avalanche] Use the proof registration state in net processing
Summary

This simplifies the logic when receiving an avalanche proof.

Ref T1854.

Depends on D10773.

Test Plan
ninja check-extended

Diff Detail

Repository
rABC Bitcoin ABC
Branch
avalanche_registration_state_net_processing
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 17806
Build 35438: Build Difflint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 35437: arc lint + arc unit

Event Timeline

Fabien requested review of this revision.Jan 6 2022, 13:54

Message change applies to an extended test

Fabien edited the test plan for this revision. (Show Details)
tyler-smith added a subscriber: tyler-smith.

I like this change; in particular, I think it's more understandable with the explicit check for the invalid state.

This revision is now accepted and ready to land.Jan 17 2022, 02:57