Page MenuHomePhabricator
Differential D6851

[avalanche] Kill PeerManager::addPeer
ClosedPublic
Actions

Authored by deadalnix on Jul 8 2020, 00:20.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC43b890a36ae9: [avalanche] Kill PeerManager::addPeer
Summary

This removes part of the Peer based API. It is replaces by a Proof based API: getPeer.

Depends on D6850

Test Plan
ninja all check-avalanche

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

deadalnix created this revision.Jul 8 2020, 00:20
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 8 2020, 00:20
deadalnix requested review of this revision.Jul 8 2020, 00:20
Harbormaster completed remote builds in B11895: Diff 22064.Jul 8 2020, 00:23
Fabien accepted this revision.Jul 8 2020, 09:26
This revision is now accepted and ready to land.Jul 8 2020, 09:26
deadalnix retitled this revision from Kill PeerManager::addPeer to [avalanche] Kill PeerManager::addPeer.Jul 8 2020, 14:54
Closed by commit rABC43b890a36ae9: [avalanche] Kill PeerManager::addPeer (authored by deadalnix). · Explain WhyJul 8 2020, 14:55
This revision was automatically updated to reflect the committed changes.
deadalnix added a commit: rABC43b890a36ae9: [avalanche] Kill PeerManager::addPeer.

Revision Contents
Changeset List

PathSize
src/
avalanche/
2 lines
10 lines
test/
35 lines

Diff 22078

View Options

src/avalanche/peermanager.h

Loading...
View Options

src/avalanche/peermanager.cpp

Loading...
View Options

src/avalanche/test/peermanager_tests.cpp

Loading...