Page MenuHomePhabricator
Differential D10663

[avalanche] Add a method to check if a proof has conflicts
AbandonedPublic
Actions

Authored by Fabien on Dec 10 2021, 14:30.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Maniphest Tasks
Restricted Maniphest Task
Summary

This is true if the proof is in the conflicting prool, or if it is in the valid pool and has any utxo that conflicts with a proof from the conflicting pool. This is the kind of proof we want to start voting on.

Ref T1854.

Depends on D10658.

Test Plan
ninja all check-avalanche

Diff Detail

Event Timeline

Fabien created this revision.Dec 10 2021, 14:30
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 10 2021, 14:30
Fabien requested review of this revision.Dec 10 2021, 14:30
Harbormaster completed remote builds in B17610: Diff 31356.Dec 10 2021, 14:45
deadalnix requested changes to this revision.Dec 10 2021, 16:35
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
src/avalanche/peermanager.h
261

Wouldn't you need at least two things to have a conflict?

This revision now requires changes to proceed.Dec 10 2021, 16:35
Fabien abandoned this revision.Feb 9 2022, 13:31

Revision Contents
Changeset List

PathSize
src/
avalanche/
1 line
21 lines
test/
58 lines

Diff 31356

View Options

src/avalanche/peermanager.h

Loading...
View Options

src/avalanche/peermanager.cpp

Loading...
View Options

src/avalanche/test/peermanager_tests.cpp

Loading...