Differential D5991

More include fixes
ClosedPublic
Actions

Authored by Fabien on May 7 2020, 08:49.

Details

Reviewers

deadalnix

Group Reviewers

Restricted Project

Commits

rSTAGING321d0956bfda: More include fixes
rABC321d0956bfda: More include fixes

Summary

The core_memusage.h header is unused by coins.h but is required for
txmempool.h, and was indirectly included via coins.h.

The blockhash.h is missing from coins.h and was indirectly included by
core_memusage.h.

Test Plan

ninja all check

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.May 7 2020, 08:49

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 7 2020, 08:49

Fabien requested review of this revision.May 7 2020, 08:49

Build Bitcoin-ABC / Diffs / Diff Testing (build-diff) started.

Build Bitcoin-ABC / Diffs / Diff Testing (build-diff) passed.

Harbormaster completed remote builds in B10631: Diff 19782.May 7 2020, 09:00

deadalnix accepted this revision.May 7 2020, 14:26

This revision is now accepted and ready to land.May 7 2020, 14:26

Closed by commit rABC321d0956bfda: More include fixes (authored by Fabien). · Explain WhyMay 7 2020, 14:46

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC321d0956bfda: More include fixes.

Fabien added a commit: rSTAGING321d0956bfda: More include fixes.May 7 2020, 14:50

Revision Contents
Changeset List

Path

Size

src/

coins.h

3 lines

txmempool.h

1 line

Diff 19792

View Options

src/coins.h