Differential D8538

validation: introduce unused ChainstateManager
ClosedPublic
Actions

Authored by Fabien on Nov 27 2020, 15:37.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC4946c4f34ab6: validation: introduce unused ChainstateManager

Summary

ChainstateManager is responsible for creating and managing multiple
chainstates, and will provide a high-level interface for accessing the
appropriate chainstate based upon a certain use.

Partial backport (2/5) of core PR17737:
https://github.com/bitcoin/bitcoin/pull/17737/commits/89cdf4d5692d396b8c7177b3918aa9dab07f9624

Depends on D8537.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Nov 27 2020, 15:37

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 27 2020, 15:37

Fabien requested review of this revision.Nov 27 2020, 15:37

teamcity edited the summary of this revision. (Show Details)Nov 27 2020, 15:37

Harbormaster completed remote builds in B14227: Diff 26063.Nov 27 2020, 15:50

Fabien added a child revision: D8539: use ChainstateManager to initialize chainstate.Nov 27 2020, 15:59

PiRK accepted this revision.Nov 27 2020, 16:58

This revision is now accepted and ready to land.Nov 27 2020, 16:58

Closed by commit rABC4946c4f34ab6: validation: introduce unused ChainstateManager (authored by James O'Beirne <james.obeirne@gmail.com>, committed by Fabien). · Explain WhyNov 30 2020, 08:02

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC4946c4f34ab6: validation: introduce unused ChainstateManager.

Revision Contents
Changeset List

Path

Size

src/

validation.h

113 lines

validation.cpp

90 lines

Diff 26090

View Options

src/validation.h