Skip to content

Commit

Permalink
More include fixes
Browse files Browse the repository at this point in the history
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

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D5991
  • Loading branch information
ftrader authored and cculianu committed Jun 7, 2020
1 parent e3cb413 commit 79e8301
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
3 changes: 1 addition & 2 deletions src/coins.h
Expand Up @@ -7,11 +7,10 @@
#define BITCOIN_COINS_H

#include <compressor.h>
#include <core_memusage.h>
#include <crypto/siphash.h>
#include <memusage.h>
#include <primitives/blockhash.h>
#include <serialize.h>
#include <uint256.h>

#include <cassert>
#include <cstdint>
Expand Down
1 change: 1 addition & 0 deletions src/txmempool.h
Expand Up @@ -8,6 +8,7 @@

#include <amount.h>
#include <coins.h>
#include <core_memusage.h>
#include <crypto/siphash.h>
#include <indirectmap.h>
#include <primitives/transaction.h>
Expand Down
1 change: 1 addition & 0 deletions src/validation.h
Expand Up @@ -13,6 +13,7 @@

#include <amount.h>
#include <blockfileinfo.h>
#include <chain.h>
#include <coins.h>
#include <consensus/consensus.h>
#include <flatfile.h>
Expand Down

0 comments on commit 79e8301

Please sign in to comment.