Differential D9038

[net processing] Move ProcessHeadersMessage to PeerManager
ClosedPublic
Actions

Authored by Fabien on Jan 22 2021, 13:05.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC6a28c545f1bd: [net processing] Move ProcessHeadersMessage to PeerManager

Summary

Partial backport (4/9) of core PR19791:
https://github.com/bitcoin/bitcoin/pull/19791/commits/d7778351bf60a21925a97b7fc4e9df5541b6d995

Depends on D9037.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

PR19791_p4

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 14952
Build 29856: Build Diff	lint-circular-dependencies · build-without-wallet · build-diff · build-clang-tidy · build-debug · build-clang
Build 29855: arc lint + arc unit

Event Timeline

Fabien created this revision.Jan 22 2021, 13:05

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 22 2021, 13:05

Fabien requested review of this revision.Jan 22 2021, 13:05

teamcity edited the summary of this revision. (Show Details)Jan 22 2021, 13:06

Harbormaster completed remote builds in B14952: Diff 27262.Jan 22 2021, 13:16

PiRK accepted this revision.Jan 22 2021, 13:34

This revision is now accepted and ready to land.Jan 22 2021, 13:34

Fabien added a child revision: D9043: [net processing] Move MaybePunishNodeForBlock into PeerManager.Jan 22 2021, 15:56

Closed by commit rABC6a28c545f1bd: [net processing] Move ProcessHeadersMessage to PeerManager (authored by John Newbery <john@johnnewbery.com>, committed by Fabien). · Explain WhyJan 23 2021, 15:05

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC6a28c545f1bd: [net processing] Move ProcessHeadersMessage to PeerManager.

Revision Contents
Changeset List

Path

Size

src/

net_processing.h

6 lines

net_processing.cpp

30 lines

Diff 27262

View Options

src/net_processing.h