Skip to content

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