Correct LoadWallet() return value (false -> DB_LOAD_OK) #1706

pull xanatos wants to merge 1 commits into bitcoin:master from xanatos:patch-6 changing 1 files +1 −1

xanatos commented at 11:43 AM on August 23, 2012: none

Equivalent code. (false == 0 == DB_LOAD_OK)

Update src/wallet.cpp

Equivalent code. (false == 0 == DB_LOAD_OK)

33d4e3ee92

jgarzik commented at 1:39 PM on August 23, 2012: contributor

Code change is correct. This change needs a better change description though. The one-line summary is "update src/wallet.cpp" which is not acceptable. Use something like "correct LoadWallet() return value"
BitcoinPullTester commented at 1:52 PM on August 24, 2012: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/33d4e3ee9233de60f93a4f2d86c9398816f5c7e5 for binaries and test log.
jgarzik commented at 4:02 PM on September 4, 2012: contributor

Ping? Improve commit message, please?
xanatos commented at 7:59 PM on September 4, 2012: none

@jgarzik woops... I corrected the name of the pull instead of the name of the commit :-)
xanatos commented at 8:33 AM on September 5, 2012: none

Woops... Reopened another pull with the correct code :-(
xanatos closed this on Sep 5, 2012
luke-jr referenced this in commit c1124277c5 on Sep 10, 2012
owlhooter referenced this in commit 312663b4be on Oct 10, 2018
KolbyML referenced this in commit c9f16d09ad on Dec 5, 2020
DrahtBot locked this on Sep 8, 2021

Contributors