The principle goal here is to make CWalletDB an internal implementation detail of CWallet.
The only remaining reference to CWalletDB in the API are for accounting tests, which can be cleaned up in a future PR.
[Wallet] Improve Wallet API Encapsulation #8055
pull pstratem wants to merge 7 commits into bitcoin:master from pstratem:2016-05-14-wallet-api-cleanup changing 9 files +143 −143-
pstratem commented at 3:41 AM on May 15, 2016: contributor
-
Move BackupWallet to CWallet::BackupWallet 01b9b217e8
-
Remove pwalletdb parameter from CWallet::AddToWallet 203f5ddbb5
- pstratem force-pushed on May 15, 2016
-
Create and use CWallet::Move 1094ee2a6c
- pstratem force-pushed on May 15, 2016
-
Create and use CWallet::ReadAccount CWallet::WriteAccount ace28ea310
- pstratem force-pushed on May 15, 2016
- pstratem renamed this:
Move BackupWallet to CWallet::BackupWallet
[WIP][Wallet] Improve Wallet API Encapsulation
on May 15, 2016 - MarcoFalke added the label Refactoring on May 15, 2016
-
MarcoFalke commented at 11:39 AM on May 15, 2016: member
Concept ACK
-
Move GetAccountBalance to CWallet::GetAccountBalance dd6e58d9bc
-
Remove references to pwalletMain inside CWallet::* methods. 6953e5373d
- pstratem renamed this:
[WIP][Wallet] Improve Wallet API Encapsulation
[Wallet] Improve Wallet API Encapsulation
on May 16, 2016 - pstratem force-pushed on May 16, 2016
-
Require CWalletDB* parameter to CWallet::IncOrderPosNext != NULL. ae411ec24a
- pstratem force-pushed on May 16, 2016
-
pstratem commented at 11:06 PM on May 16, 2016: contributor
Going to reorganize these changes
- pstratem closed this on May 16, 2016
- bitcoin locked this on Sep 8, 2021
Contributors
Labels