Page MenuHomePhabricator

[avalanche] Remove PeerManager::addNodeToPeer
ClosedPublic

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

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC216aa09cb42b: [avalanche] Remove PeerManager::addNodeToPeer
Summary

Replace it by a proof based API.

Depends on D6845 and D6848

Test Plan
ninja all check-avalanche

Event Timeline

Fabien requested changes to this revision.Jul 8 2020, 09:23
Fabien added a subscriber: Fabien.
Fabien added inline comments.
src/avalanche/processor.cpp
339

should be score

340

Nit: you can return directly

This revision now requires changes to proceed.Jul 8 2020, 09:23
This revision is now accepted and ready to land.Jul 8 2020, 14:54
This revision was landed with ongoing or failed builds.Jul 8 2020, 14:54
This revision was automatically updated to reflect the committed changes.