Permalink
Browse files

Merge #8172: Fix two warnings for comparison between signed and unsigned

77f63a4 Fix two warnings for comparison between signed and unsigned (Pieter Wuille)
  • Loading branch information...
laanwj committed Jun 9, 2016
2 parents f0299d8 + 77f63a4 commit 7e6dd7bee4791e4951870a0200b60540774e4972
Showing with 2 additions and 2 deletions.
  1. +1 −1 src/wallet/rpcwallet.cpp
  2. +1 −1 src/wallet/wallet.cpp
View
@@ -2457,7 +2457,7 @@ UniValue fundrawtransaction(const UniValue& params, bool fHelp)
if (origTx.vout.size() == 0)
throw JSONRPCError(RPC_INVALID_PARAMETER, "TX must have at least one output");
if (changePosition != -1 && (changePosition < 0 || changePosition > origTx.vout.size()))
if (changePosition != -1 && (changePosition < 0 || (unsigned int)changePosition > origTx.vout.size()))
throw JSONRPCError(RPC_INVALID_PARAMETER, "changePosition out of bounds");
CMutableTransaction tx(origTx);
View
@@ -2232,7 +2232,7 @@ bool CWallet::CreateTransaction(const vector<CRecipient>& vecSend, CWalletTx& wt
// Insert change txn at random position:
nChangePosInOut = GetRandInt(txNew.vout.size()+1);
}
else if (nChangePosInOut > txNew.vout.size())
else if ((unsigned int)nChangePosInOut > txNew.vout.size())
{
strFailReason = _("Change index out of range");
return false;

0 comments on commit 7e6dd7b

Please sign in to comment.