refactor: move BDB (bitdb / db.h) interaction from init.cpp to wallet.cpp #5758
pull jonasschnelli wants to merge 1 commits into bitcoin:master from jonasschnelli:2015/02/wallet_refactoring changing 3 files +72 −44-
jonasschnelli commented at 8:21 pm on February 4, 2015: contributorthis will remove db.h from init.cpp
-
jtimon commented at 7:21 pm on February 6, 2015: contributorut ACK
-
laanwj added the label Improvement on Feb 19, 2015
-
jgarzik commented at 2:03 pm on March 11, 2015: contributorut ACK
-
laanwj commented at 9:37 am on March 18, 2015: memberNice! Going to test.
-
refactor: move bdb (bitdb) interaction from init.cpp to wallet.cpp
this will remove db.h from init.cpp
-
in src/init.cpp: in 4d64d99e82 outdated
873- InitWarning(msg); 874- } 875- if (r == CDBEnv::RECOVER_FAIL) 876- return InitError(_("wallet.dat corrupt, salvage failed")); 877- } 878+ string warningString;
laanwj commented at 9:42 am on March 18, 2015:Nit: please usestd::string
. We’d like to get rid of the globalusing namespace
s at some point.
jonasschnelli commented at 1:14 pm on March 18, 2015:Okay. Updated. There are multiple string instantiations without thestd::
namespace within init.cpp. I don’t touch these (as well as the one i code-moved) for now. -
jonasschnelli force-pushed on Mar 18, 2015
-
laanwj referenced this in commit f3948a30cd on Mar 20, 2015
-
laanwj commented at 3:31 pm on March 20, 2015: memberMerged via 2bb1c87700a80f0b20b4b2cbd68a0d82a18dbcc0 (trivial header conflict fixed)
-
laanwj closed this on Mar 20, 2015
-
Warrows referenced this in commit 73f711cc2c on Oct 14, 2019
-
Warrows referenced this in commit 7b441e53a4 on Nov 23, 2019
-
MarcoFalke locked this on Sep 8, 2021
jonasschnelli
jtimon
jgarzik
laanwj
Labels
Refactoring