make WalletTxToJSON into a method of CWalletTx #2069

pull mikegogulski wants to merge 1 commits into bitcoin:master from mikegogulski:bf484901c9ab74f4b93f014660728ae63f4a2710 changing 3 files +25 −22

mikegogulski commented at 5:33 PM on December 4, 2012: none

start of abstraction/encapsulation fixes in the RPC interface
make WalletTxToJSON into a method of CWalletTx bf484901c9
sipa commented at 5:34 PM on December 4, 2012: member

NAK. I don't think the wallet code should depend on a JSON library.
mikegogulski commented at 5:40 PM on December 4, 2012: none

Hm, true.
mikegogulski closed this on Dec 4, 2012
BitcoinPullTester commented at 6:18 PM on December 4, 2012: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/bf484901c9ab74f4b93f014660728ae63f4a2710 for binaries and test log.
DrahtBot locked this on Sep 8, 2021

Contributors

mikegogulski

sipa

BitcoinPullTester