Page MenuHomePhabricator

[avalanche] Slightly reduce cs_main scope in registerVotes
ClosedPublic

Authored by Fabien on Oct 27 2021, 15:29.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Maniphest Tasks
Restricted Maniphest Task
Commits
rABC3e2837934f84: [avalanche] Slightly reduce cs_main scope in registerVotes
Summary

This is a follow-up to D10379. Since IsWorthPolling() also requires the lock, the WITH_LOCK() macro is not used here. There is no change in behavior.

Ref T1854.

Depends on D10379.

Test Plan
ninja all check-avalanche

Diff Detail

Repository
rABC Bitcoin ABC
Branch
avalanche_reduce_csmain_scope_registervotes
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 17121
Build 34074: Build Difflint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 34073: arc lint + arc unit