Page MenuHomePhabricator

Move finalization suck as reconsiderblock actualy reconsider blocks

Authored by deadalnix on Thu, Nov 22, 16:53.



Right now, if we reconsider a block that has been excluded due to finalization, this block will immediately get excluded again and not actually be reconsidered.

To ensure it is, move the finalization to the last common ancestor betweent he finalized block and the reconsidered block before reconsidering.

Test Plan

Added integration test.

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

deadalnix created this revision.Thu, Nov 22, 16:53
Herald added a reviewer: Restricted Project. · View Herald TranscriptThu, Nov 22, 16:53
Fabien accepted this revision.Thu, Nov 22, 17:25
This revision is now accepted and ready to land.Thu, Nov 22, 17:25
deadalnix updated this revision to Diff 6019.Thu, Nov 22, 17:36

Add an wrapper to check for finalization.

This revision was automatically updated to reflect the committed changes.