Monitor incoming transactions for spends by (a copy of) your own wallet #157

pull sipa wants to merge 1 commits into bitcoin:master from sipa:spendbycopy changing 1 files +11 −35

sipa commented at 8:49 AM on April 14, 2011: member

See here: http://www.bitcoin.org/smf/index.php?topic=3638.msg52145#msg52145
Monitor incoming transactions for spends by (a copy of) your own wallet 4a97b66796

in main.cpp:None in 46bbf50dbe outdated

 203 | @@ -204,6 +204,25 @@ bool AddToWalletIfFromMe(const CTransaction& tx, const CBlock* pblock)
 204 |      return true;
 205 |  }
 206 |  
 207 | +bool AddToWalletIfInvolvingMe(const CTransaction& tx, const CBlock* pblock)
 208 | +{
 209 | +    uint256 hash = tx.GetHash();
 210 | +    if (mapWallet.count(hash)) false;

gavinandresen commented at 3:22 PM on April 15, 2011:

return false? Or is this dead code?

sipa commented at 5:08 PM on April 15, 2011:

Return false indeed - nice catch. Fixed.

sipa commented at 8:08 PM on April 17, 2011: member

Reworked a bit, AddToWalletIfMine() and AddToWalletIfFromMe() and now merged into AddToWalletIfInvolvingMe(), simplifying things further.
gavinandresen commented at 3:41 PM on April 20, 2011: contributor

Pulled via cherry-pick
gavinandresen closed this on Apr 20, 2011
sipa referenced this in commit 94ca214e3d on Dec 18, 2014
sipa referenced this in commit 855f50ab33 on Dec 23, 2014
sipa referenced this in commit 7873633b57 on Jan 5, 2015
glv2 referenced this in commit f603af292b on Apr 22, 2015
TheBlueMatt referenced this in commit 582b2934e6 on Oct 20, 2015
rebroad referenced this in commit fd4a7032e2 on Dec 7, 2016
deadalnix referenced this in commit 2a5a3f44b7 on Dec 13, 2016
deadalnix referenced this in commit 3c0f246e7e on Jan 19, 2017
destenson referenced this in commit 0a5ef951b5 on Nov 18, 2017
lateminer referenced this in commit 518aec79e5 on Dec 9, 2017
classesjack referenced this in commit 47ec05df66 on Jan 2, 2018
attilaaf referenced this in commit 85b2110e14 on Jan 13, 2020
KolbyML referenced this in commit 0157bbc69e on Aug 1, 2020
rajarshimaitra referenced this in commit d68aad5d7a on Mar 23, 2021
cryptapus referenced this in commit 089c51d0c4 on May 3, 2021
DrahtBot locked this on Sep 8, 2021

Contributors

sipa

gavinandresen

Linked (view graph)

#25323 miniscript fuzzer failing after qa-assets update