diff options
author | fanquake <fanquake@gmail.com> | 2021-06-29 11:41:02 +0800 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2021-06-29 12:41:06 +0800 |
commit | e0face923533a1925a997228fc0c19117b19fd50 (patch) | |
tree | ade5f89622acd8fc7bc5d78f92985c4d1f8d150b /.editorconfig | |
parent | 8071ec179d75cefd41dac2a9a248eaf38054b85d (diff) | |
parent | 8888cf45f5e45b38cb830f9c94cafbf622e1fe5f (diff) |
Merge bitcoin/bitcoin#22358: Remove unused wallet pointer from wallet signals
8888cf45f5e45b38cb830f9c94cafbf622e1fe5f Remove unused wallet pointer from NotifyAddressBookChanged (MarcoFalke)
faf36403038afb3df3ddd963bd6c352d3eff4da8 Remove unused wallet pointer from NotifyTransactionChanged signal (MarcoFalke)
Pull request description:
The signals are members of the wallet, so passing the pointer would be redundant even if it was used.
Also, fix `with` -> `without`, which was forgotten in commit ca4cf5cff6f.
ACKs for top commit:
jonatack:
Code review ACK 8888cf45f5e45b38cb830f9c94cafbf622e1fe5f also verified with/without lock cs_wallet status for each of the two functions and debian clang 11 debug build clean
promag:
Code review ACK 8888cf45f5e45b38cb830f9c94cafbf622e1fe5f.
theStack:
Code review ACK 8888cf45f5e45b38cb830f9c94cafbf622e1fe5f
Tree-SHA512: e3b80931ce9bcb05213619f5435ac7c21d3c7848643950a70db610902bd1803c92bb75e501d46b0e519bc576901f160e088e8882c4f1adce892a80df565f897b
Diffstat (limited to '.editorconfig')
0 files changed, 0 insertions, 0 deletions