Skip to content

Commit

Permalink
Use z = std::max(x - y, 0); instead of z = x - y; if (z < 0) z = 0;
Browse files Browse the repository at this point in the history
  • Loading branch information
practicalswift committed Feb 7, 2017
1 parent b709fe7 commit a47da4b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
6 changes: 1 addition & 5 deletions src/addrman.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -53,11 +53,7 @@ bool CAddrInfo::IsTerrible(int64_t nNow) const
double CAddrInfo::GetChance(int64_t nNow) const
{
double fChance = 1.0;

int64_t nSinceLastTry = nNow - nLastTry;

if (nSinceLastTry < 0)
nSinceLastTry = 0;
int64_t nSinceLastTry = std::max<int64_t>(nNow - nLastTry, 0);

// deprioritize very recent attempts away
if (nSinceLastTry < 60 * 10)
Expand Down
4 changes: 1 addition & 3 deletions src/qt/coincontroldialog.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -563,9 +563,7 @@ void CoinControlDialog::updateLabels(WalletModel *model, QDialog* dialog)
}

// after fee
nAfterFee = nAmount - nPayFee;
if (nAfterFee < 0)
nAfterFee = 0;
nAfterFee = std::max<CAmount>(nAmount - nPayFee, 0);
}

// actually update labels
Expand Down

0 comments on commit a47da4b

Please sign in to comment.