make vRPCCommands comment match property names #2348

Merged
merged 2 commits into from Mar 11, 2013

Conversation

Projects
None yet
4 participants
Contributor

freewil commented Mar 7, 2013

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 freewil closed this Mar 7, 2013

@freewil freewil reopened this Mar 7, 2013

Member

luke-jr commented Mar 7, 2013

Suggest renaming "unlocked" at the same time, since it can be confused with wallet encryption lock/unlock.. "threadsafe"?

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

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/10ef3611697cdf867730689f6968ba48adc4d6e2 for binaries and test log.

gavinandresen added a commit that referenced this pull request Mar 11, 2013

Merge pull request #2348 from freewil/crpccommand-table
make vRPCCommands comment match property names

@gavinandresen gavinandresen merged commit bd80c66 into bitcoin:master Mar 11, 2013

laudney pushed a commit to reddcoin-project/reddcoin that referenced this pull request Mar 19, 2014

Merge pull request #2348 from freewil/crpccommand-table
make vRPCCommands comment match property names
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment