diff options
author | MarcoFalke <falke.marco@gmail.com> | 2020-08-14 15:11:50 +0200 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2020-08-14 15:12:44 +0200 |
commit | 30dd562fd2c58536fa026fff9853b8d825216b01 (patch) | |
tree | 06f6902b5bf5dfa4b0249c3aafef6690cd956f3b /ci/test/00_setup_env_mac_host.sh | |
parent | 1a43cd3f747f04eda32389689c006f3646b173e7 (diff) | |
parent | 0e279fe4899beae8630264ef1fe420dd71f29d5d (diff) | |
download | bitcoin-30dd562fd2c58536fa026fff9853b8d825216b01.tar.xz |
Merge #19457: wallet: Cleanup wallettool salvage and walletdb extraneous declarations
0e279fe4899beae8630264ef1fe420dd71f29d5d walletdb: Remove unused static functions from walletdb.h (Andrew Chow)
9f536d4fe949666f14a0bf5b814522cecde71f56 wallettool: Have RecoverDatabaseFile return errors and warnings (Andrew Chow)
06e263a4e368671ebb4e4a77c1447ebd5104a488 Call RecoverDatabaseFile directly from wallettool (Andrew Chow)
Pull request description:
Followup to #19324 addressing some comments.
Removes the `SalvageWallet` function in wallettool and instead directly calls `RecoverDatabaseFile` as suggested in https://github.com/bitcoin/bitcoin/pull/19324#discussion_r450379596
Removes the `LogPrintf`s and `tfm::format`s in `RecoverDatabaseFile` as noted in https://github.com/bitcoin/bitcoin/pull/19324#discussion_r448027237
Removes the declarations of `VerifyEnvironment` and `VerifyDatabaseFile` that were forgotten in `walletdb.h` as noted in https://github.com/bitcoin/bitcoin/pull/19324#issuecomment-654389079
ACKs for top commit:
meshcollider:
Code review ACK 0e279fe4899beae8630264ef1fe420dd71f29d5d
ryanofsky:
Code review ACK 0e279fe4899beae8630264ef1fe420dd71f29d5d, just dropped last commit
Tree-SHA512: ffd01f30536c2eab4bf40ba363c3ea916ecef3c8f0c5262040b40498776ffb00f95240204a40e38415d6931800851d0a3fa63ee91efc1d329b60ac317da0363d
Diffstat (limited to 'ci/test/00_setup_env_mac_host.sh')
0 files changed, 0 insertions, 0 deletions