Correct mistakes in #4377 and #4796 #5170

pull jtimon wants to merge 2 commits into bitcoin:master from jtimon:pow3 changing 6 files +20 −26
  1. jtimon commented at 8:15 pm on October 29, 2014: contributor
    1. In c2c02f3fa99385f5a5be722fda6f71522c93bdaa, UpdateTime() should have moved to miner.cpp directly instead of pow.o.
    2. After #4796, pow.cpp doesn’t depend on main.h but on chain.h.
    3. In b343c1a1e34f851e70649ad1f49855a7d878f9ef, there was no need to conserve CBlockIndex::GetBlockWork() as facade.

    I’m sorry and I want to fix it.

  2. MOVEONLY: Move void UpdateTime() from pow.o to miner.o (plus fix include main.h -> chain.h) 22c4272bf4
  3. CBlockIndex::GetBlockWork() + GetProofIncrement(nBits) -> GetBlockProof(CBlockIndex) 092b58d13d
  4. laanwj added the label Improvement on Oct 30, 2014
  5. sipa commented at 7:02 pm on November 3, 2014: member
    utACK
  6. laanwj merged this on Nov 21, 2014
  7. laanwj closed this on Nov 21, 2014

  8. laanwj referenced this in commit f2ada138c2 on Nov 21, 2014
  9. MarcoFalke locked this on Sep 8, 2021


jtimon sipa

Labels
Refactoring


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 10:12 UTC

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