HomePhabricator

[move-only] Match ordering of CAddrMan declarations and definitions

Description

[move-only] Match ordering of CAddrMan declarations and definitions

Summary:

Also move `Check` and `ForceCheckAddrman` to be after the `FunctionName_` functions.

Also move Clear() at the end together with MakeDeterministic(), as both these methods don't exist in core's codebase.

Partial backport of core#22950:
https://github.com/bitcoin/bitcoin/pull/22950/commits/f2e5f38f09ee40933f752680fe7d75ee8e529fae

Depends on D12327.

Test Plan:

ninja all check-all

Reviewers: #bitcoin_abc, PiRK

Reviewed By: #bitcoin_abc, PiRK

Differential Revision: https://reviews.bitcoinabc.org/D12328

Details

Provenance
FabienAuthored on Oct 20 2022, 08:05
FabienPushed on Oct 21 2022, 08:34
Reviewer
Restricted Project
Differential Revision
D12328: [move-only] Match ordering of CAddrMan declarations and definitions
Parents
rABC9e405b4d2a3b: [move-only] Move CAddrMan function definitions to cpp
Branches
Unknown
Tags
Unknown