Skip to content

bench: Add missing pow.h header#13530

Merged
laanwj merged 1 commit intobitcoin:masterfrom
laanwj:2018_06_add_missing_header
Jun 24, 2018
Merged

bench: Add missing pow.h header#13530
laanwj merged 1 commit intobitcoin:masterfrom
laanwj:2018_06_add_missing_header

Conversation

@laanwj
Copy link
Member

@laanwj laanwj commented Jun 24, 2018

Fix a build error introduced in #13219.

.../bitcoin/src/bench/block_assemble.cpp:42:13:error: use of undeclared identifier 'CheckProofOfWork'
    while (!CheckProofOfWork(block->GetHash(), block->nBits, Params().GetConsensus())) {

Fix a build error introduced in bitcoin#13219.

```
.../bitcoin/src/bench/block_assemble.cpp:42:13:error: use of undeclared identifier 'CheckProofOfWork'
    while (!CheckProofOfWork(block->GetHash(), block->nBits, Params().GetConsensus())) {
```
@laanwj laanwj added the Tests label Jun 24, 2018
@sipa
Copy link
Member

sipa commented Jun 24, 2018

ACK cec84c2, build is broken without it.

@laanwj laanwj merged commit cec84c2 into bitcoin:master Jun 24, 2018
laanwj added a commit that referenced this pull request Jun 24, 2018
cec84c2 bench: Add missing pow.h header (Wladimir J. van der Laan)

Pull request description:

  Fix a build error introduced in #13219.

  ```
  .../bitcoin/src/bench/block_assemble.cpp:42:13:error: use of undeclared identifier 'CheckProofOfWork'
      while (!CheckProofOfWork(block->GetHash(), block->nBits, Params().GetConsensus())) {
  ```

Tree-SHA512: 6ef78011fd7bd82859530b2676ba533b15db6c753cb84a1286b7e26ed6f3d2064769e42900b2479fe6c460411f742e2baed716db5eac06d3bc455e08c290a087
@maflcko
Copy link
Member

maflcko commented Jun 24, 2018

ACK, seems this was a silent merge conflict with 5b35b92

@maflcko
Copy link
Member

maflcko commented Jun 24, 2018

Also my lack of using iwyu

@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants