Page MenuHomePhabricator

Merge #14784: qt: Use WalletModel* instead of the wallet name as map key

Authored by jasonbcox on Nov 27 2019, 20:07.



91b0c5b09616bb8c5ef8107efb432ce4c7c9a383 qt: Use WalletModel* instead of wallet name in console window (João Barbosa)
b2ce86c3ad79db49ddf06207230512300ec11156 qt: Use WalletModel* instead of wallet name in main window (João Barbosa)
d2a1adffebee86763fafc6f3ed2722f7038654c7 qt: Factor out WalletModel::getDisplayName() (João Barbosa)

Pull request description:

This a small refactor that doesn't change behavior. This is also necessary if in the future we allow renaming wallets.

Tree-SHA512: 1820d0ff28e84b1d862097f1f55b52f94520fa50c9b1939d235a448a48159748c3bbf99b19e4cb1ff4f91efc008c0971b4c25a91f645f9d43792c8aeaa93cf9e

Backport of Core PR14784

Completes T698

Test Plan

ninja check

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jasonbcox created this revision.Nov 27 2019, 20:07
jasonbcox edited the summary of this revision. (Show Details)Nov 27 2019, 20:18
Fabien accepted this revision.Nov 27 2019, 20:28
This revision is now accepted and ready to land.Nov 27 2019, 20:28