Page MenuHomePhabricator

AreOnTheSameFork: don't actually need to find common ancestor
ClosedPublic

Authored by markblundeberg on Jan 26 2020, 05:46.

Details

Summary

We don't actually need to know what is the common ancestor (which may
require iterating over many blocks below pa and pb). It's enough to
just perform the initial phase from LastCommonAncestor, which is
what this new code does.

Test Plan

ninja check-all

Diff Detail

Repository
rABC Bitcoin ABC
Branch
AreOnTheSameFork_simplify
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 9142
Build 16242: Default Diff Build & Tests
Build 16241: arc lint + arc unit