prefer std::string over char* in CDB constructor

2nd commit:
cleanup class private and public areas in walletdb
Can also be `const std::string &strFilename`