diff options
| author | Jonas Schnelli <dev@jonasschnelli.ch> | 2020-11-12 12:23:51 +0100 |
|---|---|---|
| committer | Jonas Schnelli <dev@jonasschnelli.ch> | 2020-11-12 12:23:55 +0100 |
| commit | 9bd1316697292251ed5690390794fc64517fc86b (patch) | |
| tree | d62f351ce67c08a84e01d243f64bab16977f78eb /src/test/fuzz/process_message.cpp | |
| parent | 8a486158cbc3f18adf12d2d3b737f7f4e7399e7d (diff) | |
| parent | 241434200ec2067673d8522fee4f1228abfd8247 (diff) | |
Merge bitcoin-core/gui#120: Fix multiwallet transaction notifications
241434200ec2067673d8522fee4f1228abfd8247 refactor: qt: Use vQueueNotifications.clear() (João Barbosa)
989e579d07bb5031639060b717f7a0be15d10e29 qt: Make transaction notification queue wallet specific (João Barbosa)
7b3b2303f44031c3545651858f697a495c3ea37a move-only: Define TransactionNotification before TransactionTablePriv (João Barbosa)
Pull request description:
Currently `vQueueNotifications` holds transactions of any wallet, but the queue is dispatched on a given wallet and it assumes notifications are of that wallet.
This means that some transactions can be missed if multiple wallets are loaded.
Fix this by having a queue for each wallet.
ACKs for top commit:
jonasschnelli:
utACK 241434200ec2067673d8522fee4f1228abfd8247
hebasto:
ACK 241434200ec2067673d8522fee4f1228abfd8247, I have reviewed the code and it looks OK, I agree it can be merged.
ryanofsky:
Code review ACK 241434200ec2067673d8522fee4f1228abfd8247. Only change is dropping one commit
Tree-SHA512: 61beac5a16ed659e3a25ad145dbceafcef963aaf8f9838355298949ec2324e2bd760f59353cd251d30cf0334d8dc1642a1f3821d8a9eec092533b581f6ce86db
Diffstat (limited to 'src/test/fuzz/process_message.cpp')
0 files changed, 0 insertions, 0 deletions