make vRPCCommands comment match property names #2348

pull freewil wants to merge 2 commits into bitcoin:master from freewil:crpccommand-table changing 2 files +66 −66

freewil commented at 10:19 AM on March 7, 2013: contributor

This is a real simple change to make the comments match the property names of CRPCCommand

Here is the diff, ignoring the whitespace: https://github.com/bitcoin/bitcoin/pull/2348/files?w=1
freewil closed this on Mar 7, 2013
make vRPCCommands comment match property names dcf8b8fd9e
freewil reopened this on Mar 7, 2013
luke-jr commented at 10:33 AM on March 7, 2013: member

Suggest renaming "unlocked" at the same time, since it can be confused with wallet encryption lock/unlock.. "threadsafe"?
BitcoinPullTester commented at 11:06 AM on March 7, 2013: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/dcf8b8fd9e9fc964eda083ee82e46df58958f705 for binaries and test log.
CRPCCommand.unlocked -> CRPCCommand.threadSafe
unlocked could be confused with wallet encryption
10ef361169
BitcoinPullTester commented at 11:56 AM on March 7, 2013: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/10ef3611697cdf867730689f6968ba48adc4d6e2 for binaries and test log.
gavinandresen referenced this in commit bd80c66ac0 on Mar 11, 2013
gavinandresen merged this on Mar 11, 2013
gavinandresen closed this on Mar 11, 2013
laudney referenced this in commit 7c05d543a6 on Mar 19, 2014
DrahtBot locked this on Sep 8, 2021

Contributors