Page MenuHomePhabricator

[avalanche] Clarify the processor includes
ClosedPublic

Authored by Fabien on Jan 25 2023, 12:54.

Details

Reviewers
sdulfari
Group Reviewers
Restricted Project
Commits
rABC6cd0599cb28c: [avalanche] Clarify the processor includes
Summary

The CBlockIndex is now used in the comparator and is not forwarded, but indirectly included. The ProofRef is also indirectly included, while Proof is no longer used. This diff cleans these inclusions so it's easier to reason about.

Test Plan

With Clang and GCC:

ninja all check-avalanche

Diff Detail

Repository
rABC Bitcoin ABC
Branch
avalanche_add_missing_header
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 21868
Build 43371: Build Diffbuild-without-wallet · build-clang-tidy · lint-circular-dependencies · build-debug · build-clang · build-diff
Build 43370: arc lint + arc unit