validation: Use existing chainstate in ChainstateManager::ProcessNewBlockHeaders
ClosedPublic
Actions

Authored by PiRK on Mar 16 2022, 12:07.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCe9bcf5e92ce3: validation: Use existing chainstate in ChainstateManager::ProcessNewBlockHeaders

Summary

[META] This commit should be followed up by removing the comments and assertions meant only to show that the change is correct.

This is a backport of core#20749 [14 & 15/17]
https://github.com/bitcoin/bitcoin/pull/20749/commits/ea4fed90219be17160136313c68c06d84176af08
https://github.com/bitcoin/bitcoin/pull/20749/commits/0cdad753903640ff4240b715dec9d62f68e51407

Depends on D11185

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

PiRK created this revision.Mar 16 2022, 12:07

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 16 2022, 12:07

PiRK requested review of this revision.Mar 16 2022, 12:07

teamcity edited the summary of this revision. (Show Details)Mar 16 2022, 12:07

Harbormaster completed remote builds in B18509: Diff 32745.Mar 16 2022, 12:22

Fabien accepted this revision.Mar 16 2022, 13:18

This revision is now accepted and ready to land.Mar 16 2022, 13:18

Closed by commit rABCe9bcf5e92ce3: validation: Use existing chainstate in ChainstateManager::ProcessNewBlockHeaders (authored by Carl Dong <contact@carldong.me>, committed by PiRK). · Explain WhyMar 16 2022, 14:27

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABCe9bcf5e92ce3: validation: Use existing chainstate in ChainstateManager::ProcessNewBlockHeaders.

Revision Contents
Changeset List

Path

Size

src/

validation.cpp

16 lines

Diff 32753

View Options

validation: Use existing chainstate in ChainstateManager::ProcessNewBlockHeadersClosedPublicActions