[net processing] Move CNodeState declaration above PeerManagerImpl
ClosedPublic
Actions

Authored by PiRK on Nov 28 2023, 13:57.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC0316a790003a: [net processing] Move CNodeState declaration above PeerManagerImpl

Summary

This is a partial backport of core#24543
https://github.com/bitcoin/bitcoin/pull/24543/commits/37ecaf3e7a028486a0a1c9b717e8eb4214215805

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 25752
Build 51084: Build Diff	lint-circular-dependencies · build-clang-tidy · build-without-wallet · build-diff · build-clang · build-debug
Build 51083: arc lint + arc unit

Event Timeline

PiRK created this revision.Nov 28 2023, 13:57

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 28 2023, 13:57

PiRK requested review of this revision.Nov 28 2023, 13:57

teamcity edited the summary of this revision. (Show Details)Nov 28 2023, 13:57

PiRK added a child revision: D14862: [net processing] Move mapNodeState into PeerManagerImpl.Nov 28 2023, 14:01

Harbormaster completed remote builds in B25752: Diff 43285.Nov 28 2023, 14:14

OK because it's move only

src/net_processing.cpp
649	???

This revision is now accepted and ready to land.Nov 29 2023, 10:01

Closed by commit rABC0316a790003a: [net processing] Move CNodeState declaration above PeerManagerImpl (authored by PiRK). · Explain WhyNov 29 2023, 11:38

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC0316a790003a: [net processing] Move CNodeState declaration above PeerManagerImpl.

Revision Contents
Changeset List

Path

Size

src/

net_processing.cpp

218 lines

Diff 43285

View Options

[net processing] Move CNodeState declaration above PeerManagerImplClosedPublicActions