Belovachap/compiler warnings

[belovachap]

Address some of the compiler warnings that might have come from us.

[backpacker69]

we are likely to have softforks in future why do you remove this code?

[belovachap]

we are likely to have softforks in future why do you remove this code?

The compiler warning that prompted me to remove it:

rpc/blockchain.cpp:1053:17: warning: ‘UniValue SoftForkDesc(const string&, int, CBlockIndex*, const Consensus::Params&)’ defined but not used [-Wunused-function]
 static UniValue SoftForkDesc(const std::string &name, int version, CBlockIndex* pindex, const Consensus::Params& consensusParams)
                 ^~~~~~~~~~~~

but if you think we'll need SoftForkDesc at some point, I've added it back in :)