wallet encapsulation #2130
wallet encapsulation #2130
Conversation
CWallet::GetAccountAddresses()
src/wallet.cpp
Outdated
There was a problem hiding this comment.
Here's still a CBitcoinAddress left.
There was a problem hiding this comment.
I need to hunt that one down yet since I think it's used outside of rpc*.cpp. I'll post a new commit working on that soon.
CWallet::IsMine(CTxDestination&)
|
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/943e7768f33a7c766d1be5db8aea64a46e37253b for binaries and test log. |
src/rpcwallet.cpp
Outdated
There was a problem hiding this comment.
I saw a method returning bool and the return value being ignored, so I put in that note. Looking around the codebase now I see that it's hardly ever used, so maybe it can be ignored. Want a new commit?
|
Many of your TODO's look like todo's for yourself rather than concrete plans for changes in the source code - leave those out. If there is a concrete plan, and it's obvious: just add a commit that actually implements the change. If it's something big you'd rather leave for a follow-up pull request, a TODO in the code is fine. Also, squash some commits together. "Indentation fix" certainly doesn't require a separate commit. |
|
Okay, I stripped out the todos. Should I squash everything in this branch down to one commit in order to get it pulled? (total git noob here) |
|
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/4debb8059a3d29a88b72b2bc61a4e0f6bfc7e522 for binaries and test log. |
|
Automatic sanity-testing: FAILED MERGE, see http://jenkins.bluematt.me/pull-tester/4debb8059a3d29a88b72b2bc61a4e0f6bfc7e522 for test log. This pull does not merge cleanly onto current master |
|
Rebase needed |
|
Poke @mikegogulski The general sentiment towards these changes seems positive. Let's rebase and get this moving, or close. |
|
I've rebased this as my mikegogulski_walletencap3 branch. It does, however, create a dependency from wallet.cpp -> main.h; I don't see any obvious way to fix this, considering that IsFinalTx is not tied to any class. |
|
Closing - non-responsive. Feel free to rebase and reopen. |
5 participants
hopefully a proper rebasing of #2075
The goal here is to work toward a clean interface to the wallet object. For now this involves moving code out of rpc*.cpp which deals with wallet internals and making that code into methods on the CWallet object.