diff options
author | Pieter Wuille <pieter.wuille@gmail.com> | 2018-03-13 19:07:49 -0700 |
---|---|---|
committer | Pieter Wuille <pieter.wuille@gmail.com> | 2018-03-13 19:16:39 -0700 |
commit | 6acd8700bc0ee1d10207a362c1e07372ba274041 (patch) | |
tree | 27ab248388e4a316ce41d750098e215ba7288f44 /src/scheduler.cpp | |
parent | 18462960c0f13bd07d8f52b61e7d7bc17e991eea (diff) | |
parent | b4bc32a451720167000f59dd73ab07990f9c6b92 (diff) | |
download | bitcoin-6acd8700bc0ee1d10207a362c1e07372ba274041.tar.xz |
Merge #9680: Unify CWalletTx construction
b4bc32a451 [wallet] Get rid of CWalletTx default constructor (Russell Yanofsky)
a128bdc9e1 [wallet] Construct CWalletTx objects in CommitTransaction (Russell Yanofsky)
Pull request description:
Two commits:
- `Construct CWalletTx objects in CommitTransaction` moves a bunch of CWalletTx initialization into CWallet::CommitTransaction to dedup some code and avoid future inconsistencies in how wallet transactions are created.
- `Get rid of CWalletTx default constructor` does what is described and eliminates the possibility of empty transaction entries being inadvertently created by mapWallet[hash] accesses.
Both of these changes were originally part of #9381
Tree-SHA512: af3841c4f0539e0662d81b33c5369fc70aa06ddde1c59cb00fb21c9e4c7d9ff47f1edc5040cb463af1333838802c56b3ef875b939e2b804ee45b8e0294a4371c
Diffstat (limited to 'src/scheduler.cpp')
0 files changed, 0 insertions, 0 deletions