Differential D10876

[net processing] Move send ping message logic into function
ClosedPublic
Actions

Authored by Fabien on Jan 24 2022, 21:13.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Maniphest Tasks

Restricted Maniphest Task

Commits

rABC83bf47dcdcc8: [net processing] Move send ping message logic into function

Summary

Partial backport of core#20721:
https://github.com/bitcoin/bitcoin/pull/20721/commits/0b43b81f69ff13dbc1e893a80950f186690b4f62

Ref T1696.

Depends on D10875.

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.Jan 24 2022, 21:13

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 24 2022, 21:13

Fabien requested review of this revision.Jan 24 2022, 21:13

teamcity edited the summary of this revision. (Show Details)Jan 24 2022, 21:14

Fabien added a child revision: D10877: [net processing] Move ping timeout logic to net processing.Jan 24 2022, 21:25

Harbormaster completed remote builds in B17985: Diff 31925.Jan 24 2022, 21:28

Missing dependency, needs rebase

Rebase

Harbormaster completed remote builds in B18007: Diff 31960.Jan 25 2022, 15:46

PiRK accepted this revision.Jan 25 2022, 15:50

This revision is now accepted and ready to land.Jan 25 2022, 15:50

Closed by commit rABC83bf47dcdcc8: [net processing] Move send ping message logic into function (authored by John Newbery <john@johnnewbery.com>, committed by Fabien). · Explain WhyJan 25 2022, 16:39

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC83bf47dcdcc8: [net processing] Move send ping message logic into function.

Fabien removed a child revision: D10877: [net processing] Move ping timeout logic to net processing.Jan 25 2022, 16:45

Revision Contents
Changeset List

Path

Size

src/

net_processing.cpp

70 lines

Diff 31966

View Options

[net processing] Move send ping message logic into functionClosedPublicActions