I prefer, but do not insist on this. Might want to refactor the remaining dummy CTxUndo txundo; at some other time.
Remove obsolete reference to CValidationState from UpdateCoins. #7976
pull ghost wants to merge 1 commits into bitcoin:master from changing 4 files +8 −9-
ghost commented at 3:51 AM on April 30, 2016: none
- MarcoFalke added the label Refactoring on Apr 30, 2016
-
Remove obsolete reference to CValidationState from UpdateCoins. c8b92486c4
-
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.
dcousens commented at 1:22 AM on May 2, 2016: contributorutACK c8b9248
sipa commented at 5:00 PM on May 5, 2016: memberI like this. utACK c8b92486c4eca1287546b0683b6af3551db1fe67
laanwj commented at 9:58 AM on May 9, 2016: memberutACK c8b9248
laanwj merged this on May 9, 2016laanwj closed this on May 9, 2016laanwj referenced this in commit e29cfc48fc on May 9, 2016codablock referenced this in commit 91a7051e20 on Sep 16, 2017codablock referenced this in commit 843d8c048f on Sep 19, 2017codablock referenced this in commit 5222e9377d on Dec 21, 2017zkbot referenced this in commit 84c19b8a87 on Feb 8, 2018zkbot referenced this in commit e1f3a15fdc on Feb 8, 2018zkbot referenced this in commit eb3128ab0a on Feb 19, 2018zkbot referenced this in commit 6db10127a9 on Feb 20, 2018zkbot referenced this in commit 8487be8360 on Feb 20, 2018random-zebra referenced this in commit 5a092159f6 on Aug 5, 2020MarcoFalke locked this on Sep 8, 2021ContributorsLabels
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 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
More mirrored repositories can be found on mirror.b10c.me