Page MenuHomePhabricator

[avalanche] Release the locking requirements in AvalancheFinalizeBlock
ClosedPublic

Authored by Fabien on Jul 25 2025, 09:16.

Details

Summary

By splitting out the staking rewards we can make the whole function require cs_main instead of having to release the lock. This makes the logic easier to reason about and will make the follow up code more robust.
There is no change in behavior.

Test Plan
ninja check-extended

Diff Detail

Repository
rABC Bitcoin ABC
Branch
avalanche_avalanchefinalizedblock_csmain
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 33971
Build 67411: Build Diffbuild-without-wallet · build-debug · build-diff · lint-circular-dependencies · build-clang · build-clang-tidy
Build 67410: arc lint + arc unit