Differential D12254

move-only: Group and re-order CAddrMan members by access type
ClosedPublic
Actions

Authored by Fabien on Oct 14 2022, 13:09.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCf74cc137efb7: move-only: Group and re-order CAddrMan members by access type

Summary

Order them as public:, protected:, private:. There is no change, move only.

Completes backport of core#22025:
https://github.com/bitcoin/bitcoin/pull/22025/commits/8caf60dbbe16afa3c52574a7f6710d74c0bfd4ab

Depends on D12253.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Oct 14 2022, 13:09

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 14 2022, 13:09

Fabien requested review of this revision.Oct 14 2022, 13:09

teamcity edited the summary of this revision. (Show Details)Oct 14 2022, 13:09

Harbormaster completed remote builds in B20515: Diff 35694.Oct 14 2022, 13:27

PiRK accepted this revision.Oct 14 2022, 13:58

This revision is now accepted and ready to land.Oct 14 2022, 13:58

Fabien mentioned this in D12257: refactor: Make CAddrMan::cs non-recursive.Oct 14 2022, 14:09

Closed by commit rABCf74cc137efb7: move-only: Group and re-order CAddrMan members by access type (authored by Hennadii Stepanov <32963518+hebasto@users.noreply.github.com>, committed by Fabien). · Explain WhyOct 14 2022, 18:56

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCf74cc137efb7: move-only: Group and re-order CAddrMan members by access type.

Fabien mentioned this in rABCb8bdc0c6dd89: refactor: Make CAddrMan::cs non-recursive.Oct 14 2022, 18:59

Revision Contents
Changeset List

Path

Size

src/

addrman.h

324 lines

Diff 35712

View Options

move-only: Group and re-order CAddrMan members by access typeClosedPublicActions