extension of #1103: encapsulate mapCommands in CRPCTable #1124

pull sipa wants to merge 3 commits into bitcoin:master from sipa:rpcobj3 changing 3 files +130 −106

sipa commented at 8:45 PM on April 18, 2012: member

No description provided.
sipa cross-referenced this on Apr 18, 2012 from issue Encapsulate RPC command dispatch in an array of CRPCCommand's by jgarzik
jgarzik commented at 10:10 PM on April 18, 2012: contributor

ACK
laanwj commented at 5:21 AM on April 19, 2012: member

ACK
Encapsulate RPC command dispatch in an array of CRPCCommand's dc42bf52c1
Encapsulate mapCommands in class CRPCTable 9862229d4d
Expose CRPCTable via bitcoinrpc.h for testing e46704dd90
sipa commented at 11:41 PM on April 20, 2012: member

I added a commit that moves the definition of CRPCTable and CRPCCommand to bitcoinrpc.h, so that it does not need to be duplicated in tests/rpc_tests.cpp. I suppose it is useful for #1075 as well. re-ACK?
jgarzik commented at 11:44 PM on April 20, 2012: contributor

ACK
gavinandresen commented at 11:31 PM on April 21, 2012: contributor

ACK
sipa referenced this in commit 457661f640 on Apr 21, 2012
sipa merged this on Apr 21, 2012
sipa closed this on Apr 21, 2012
coblee referenced this in commit a5fd7c991c on Jul 17, 2012
suprnurd referenced this in commit fef8752cd0 on Dec 5, 2017
lateminer referenced this in commit ab81a0c27f on Jan 22, 2019
lateminer referenced this in commit bbb0125077 on Dec 25, 2019
lateminer referenced this in commit 7fdf25b4f7 on May 6, 2020
bitcoin locked this on Sep 8, 2021

Contributors

Milestone
0.6.1