Consensus: Consensus version of pow functions #6037

pull jtimon wants to merge 1 commits into bitcoin:master from jtimon:consensus_pow2 changing 11 files +77 −61
  1. jtimon commented at 6:42 pm on April 20, 2015: contributor

    Removes most of the undesired dependencies from the consensus pow functions. CBlockIndex is still used, but only its attributes and none of the methods. This is part of #5995. It also relates with some miner PRs (like #4793 or #5957) since this will stop those log messages in regtest, allowing those PRs to use CheckProofOfWork directly in ScanHash. I believe that’s something that at least @sipa and @21E14 will like.

    Questions:

    -Is it worth it to create miner::GetNextWorkRequiredLog() ? -Should CheckProofOfWork() receive a reference to a CValidationState to be able to give the specific reason when it returns false?

  2. jtimon force-pushed on Apr 20, 2015
  3. Consensus: Consensus version of pow functions 3dee0253b9
  4. jtimon force-pushed on Apr 20, 2015
  5. jtimon commented at 4:12 pm on April 22, 2015: contributor
    Closing for now
  6. jtimon closed this on Apr 22, 2015

  7. DrahtBot locked this on Sep 8, 2021


jtimon


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-09-29 04:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me