Common argument defaults for NODE_BLOOM stuff and -wallet

[luke-jr]

Can we stop adding more of these to fix? :p

[paveljanik]

When you are at this...

These:

init.cpp:    strUsage += HelpMessageOpt("-salvagewallet", _("Attempt to recover private keys from a corrupt wallet.dat on startup"));
init.cpp:                uiInterface.InitMessage(_("Error loading wallet.dat: Wallet corrupted"));
init.cpp:                strErrors << _("Error loading wallet.dat: Wallet corrupted") << "\n";
init.cpp:                InitWarning(_("Error reading wallet.dat! All keys read correctly, but transaction data"
init.cpp:                strErrors << strprintf(_("Error loading wallet.dat: Wallet requires newer version of %s"), _(PACKAGE_NAME)) << "\n";
init.cpp:                strErrors << _("Error loading wallet.dat") << "\n";

should probably use the selected wallet.dat file name (strWalletFile) instead of the hardcoded name. And there are other such cases.

[laanwj]

Concept ACK
-enforcenodebloom is going away soon (#6641), so I don't think it needs any fixes/improvements

[pstratem]

utACK 1fb91b3

[dcousens]

utACK 1fb91b3

[laanwj]

@paveljanik I think the -salvagewallet help message should simply say 'wallet', not 'wallet.dat'.
For the warnings and errors you're right and it'd be better to use the actual wallet file name.