Changes (reason):
- Return
voidinCallRPC(the result is unused anyway) - Reduce the
catch-scope ofstd::runtime_errortoRPCConvertValues(Code clarity and easier bug-finding) - Crash when an internal bug is detected (bugs are bad)
Changes (reason):
void in CallRPC (the result is unused anyway)catch-scope of std::runtime_error to RPCConvertValues (Code clarity and easier bug-finding)Code Review ACK fa52a86fd3acbcfc4b5ca1304c19d81df66d85d7