Remove obsolete reference to CValidationState from UpdateCoins. #7976

pull ghost wants to merge 1 commits into bitcoin:master from changing 4 files +8 −9
  1. ghost commented at 3:51 AM on April 30, 2016: none

    I prefer, but do not insist on this. Might want to refactor the remaining dummy CTxUndo txundo; at some other time.

  2. MarcoFalke added the label Refactoring on Apr 30, 2016
  3. Remove obsolete reference to CValidationState from UpdateCoins. c8b92486c4
  4. in src/main.h:None in 16dda8f3f1 outdated
     358 | @@ -359,7 +359,7 @@ bool CheckInputs(const CTransaction& tx, CValidationState &state, const CCoinsVi
     359 |                   unsigned int flags, bool cacheStore, std::vector<CScriptCheck> *pvChecks = NULL);
     360 |  
     361 |  /** Apply the effects of this transaction on the UTXO set represented by view */
     362 | -void UpdateCoins(const CTransaction& tx, CValidationState &state, CCoinsViewCache &inputs, int nHeight);
     363 | +void UpdateCoins(const CTransaction& tx, CCoinsViewCache &inputs, int nHeight);
    


    MarcoFalke commented at 4:17 PM on April 30, 2016:

    nit: If you do trivial fixes, please also do all other trivial fixes in the same line:

    diff --git a/src/main.cpp b/src/main.cpp
    index c5d2e23..7d5beab 100644
    --- a/src/main.cpp
    +++ b/src/main.cpp
    @@ -1747 +1747 @@ void static InvalidBlockFound(CBlockIndex *pindex, const CValidationState &state
    -void UpdateCoins(const CTransaction& tx, CCoinsViewCache &inputs, CTxUndo &txundo, int nHeight)
    +void UpdateCoins(const CTransaction& tx, CCoinsViewCache& inputs, CTxUndo& txundo, int nHeight)
    @@ -1773 +1773 @@ void UpdateCoins(const CTransaction& tx, CCoinsViewCache &inputs, CTxUndo &txund
    -void UpdateCoins(const CTransaction& tx, CCoinsViewCache &inputs, int nHeight)
    +void UpdateCoins(const CTransaction& tx, CCoinsViewCache& inputs, int nHeight)
    diff --git a/src/main.h b/src/main.h
    index 716378e..792747a 100644
    --- a/src/main.h
    +++ b/src/main.h
    @@ -362 +362 @@ bool CheckInputs(const CTransaction& tx, CValidationState &state, const CCoinsVi
    -void UpdateCoins(const CTransaction& tx, CCoinsViewCache &inputs, int nHeight);
    +void UpdateCoins(const CTransaction& tx, CCoinsViewCache& inputs, int nHeight);
    

    unknown commented at 7:15 PM on April 30, 2016:

    Upping the ante? Updated.

  5. dcousens commented at 1:22 AM on May 2, 2016: contributor

    utACK c8b9248

  6. sipa commented at 5:00 PM on May 5, 2016: member

    I like this. utACK c8b92486c4eca1287546b0683b6af3551db1fe67

  7. laanwj commented at 9:58 AM on May 9, 2016: member

    utACK c8b9248

  8. laanwj merged this on May 9, 2016
  9. laanwj closed this on May 9, 2016

  10. laanwj referenced this in commit e29cfc48fc on May 9, 2016
  11. codablock referenced this in commit 91a7051e20 on Sep 16, 2017
  12. codablock referenced this in commit 843d8c048f on Sep 19, 2017
  13. codablock referenced this in commit 5222e9377d on Dec 21, 2017
  14. zkbot referenced this in commit 84c19b8a87 on Feb 8, 2018
  15. zkbot referenced this in commit e1f3a15fdc on Feb 8, 2018
  16. zkbot referenced this in commit eb3128ab0a on Feb 19, 2018
  17. zkbot referenced this in commit 6db10127a9 on Feb 20, 2018
  18. zkbot referenced this in commit 8487be8360 on Feb 20, 2018
  19. random-zebra referenced this in commit 5a092159f6 on Aug 5, 2020
  20. MarcoFalke locked this on Sep 8, 2021

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: 2026-04-17 15:15 UTC

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