-usecashaddr problems when using --disable-wallet

[sickpig]

if --disable-wallet is passed to the configure script using usecashaddr will produce an error on startup.

This is due to the fact that usecashaddr is considered valid only in the presence of the wallet functionality.

Even if usecashaddr got compiled in w/ wallet disabled via a path like the one at the bottom, the parameter is not honored anyway.

diff --git a/src/allowed_args.cpp b/src/allowed_args.cpp
index 1b759f878..52e61c76a 100644
--- a/src/allowed_args.cpp
+++ b/src/allowed_args.cpp
@@ -318,7 +318,12 @@ static void addGeneralOptions(AllowedArgs &allowedArgs, HelpMessageMode mode)
         .addArg("reindex", optionalBool, _("Rebuild block chain index from current blk000??.dat files on startup"))
         .addArg("txindex", optionalBool,
             strprintf(_("Maintain a full transaction index, used by the getrawtransaction rpc call (default: %u)"),
-                    DEFAULT_TXINDEX));
+                    DEFAULT_TXINDEX))
+        .addArg(
+            "usecashaddr", optionalBool,
+            strprintf(
+                _("Use Bitcoin Cash Address for destination encoding (Activates by default Jan 14, 2017, default %u)"),
+                true));
 }
 
 static void addConnectionOptions(AllowedArgs &allowedArgs)
@@ -488,9 +493,6 @@ static void addWalletOptions(AllowedArgs &allowedArgs)
               "startup") +
                 " " +
                 _("(1 = keep tx meta data e.g. account owner and payment request information, 2 = drop tx meta data)"),
-            walletParamOptional)
-        .addArg("usecashaddr", optionalBool,
-            _("Use Bitcoin Cash Address for destination encoding (Activates by default Jan 14, 2017)"),
             walletParamOptional);
 }
 #endif