Drop ParseHashUV in favor of calling ParseHashStr

Empact · Empact · commit abd2678ac1af · 2018-06-08T10:53:38.000-07:00
The one existing call already validates get_str will
pass via checkObject.
diff --git a/src/bitcoin-tx.cpp b/src/bitcoin-tx.cpp
@@ -591,7 +591,7 @@ static void MutateTxSign(CMutableTransaction& tx, const std::string& flagStr)
             if (!prevOut.checkObject(types))
                 throw std::runtime_error("prevtxs internal object typecheck fail");
 
-            uint256 txid = ParseHashUV(prevOut["txid"], "txid");
+            uint256 txid = ParseHashStr(prevOut["txid"].get_str(), "txid");
 
             int nOut = atoi(prevOut["vout"].getValStr());
             if (nOut < 0)
diff --git a/src/core_io.h b/src/core_io.h
@@ -22,7 +22,6 @@ CScript ParseScript(const std::string& s);
 std::string ScriptToAsmStr(const CScript& script, const bool fAttemptSighashDecode = false);
 bool DecodeHexTx(CMutableTransaction& tx, const std::string& hex_tx, bool try_no_witness = false, bool try_witness = true);
 bool DecodeHexBlk(CBlock&, const std::string& strHexBlk);
-uint256 ParseHashUV(const UniValue& v, const std::string& strName);
 uint256 ParseHashStr(const std::string&, const std::string& strName);
 std::vector<unsigned char> ParseHexUV(const UniValue& v, const std::string& strName);
 
diff --git a/src/core_read.cpp b/src/core_read.cpp
@@ -160,14 +160,6 @@ bool DecodeHexBlk(CBlock& block, const std::string& strHexBlk)
     return true;
 }
 
-uint256 ParseHashUV(const UniValue& v, const std::string& strName)
-{
-    std::string strHex;
-    if (v.isStr())
-        strHex = v.getValStr();
-    return ParseHashStr(strHex, strName);  // Note: ParseHashStr("") throws a runtime_error
-}
-
 uint256 ParseHashStr(const std::string& strHex, const std::string& strName)
 {
     if (!IsHex(strHex)) // Note: IsHex("") is false

Original file line number	Diff line number	Diff line change
`@@ -160,14 +160,6 @@ bool DecodeHexBlk(CBlock& block, const std::string& strHexBlk)`
`160`	`160`	`return true;`
`161`	`161`	`}`
`162`	`162`
`163`		`-uint256 ParseHashUV(const UniValue& v, const std::string& strName)`
`164`		`-{`
`165`		`- std::string strHex;`
`166`		`- if (v.isStr())`
`167`		`- strHex = v.getValStr();`
`168`		`- return ParseHashStr(strHex, strName); // Note: ParseHashStr("") throws a runtime_error`
`169`		`-}`
`170`		`-`
`171`	`163`	`uint256 ParseHashStr(const std::string& strHex, const std::string& strName)`
`172`	`164`	`{`
`173`	`165`	`if (!IsHex(strHex)) // Note: IsHex("") is false`