diff options
author | Wladimir J. van der Laan <laanwj@gmail.com> | 2017-02-15 09:26:12 +0100 |
---|---|---|
committer | Wladimir J. van der Laan <laanwj@gmail.com> | 2017-02-15 09:27:47 +0100 |
commit | 4c69d683f22ad8bc310c901b50b8892da7a3c70c (patch) | |
tree | dd6fa87b3f96b2bf3bd39cfb3598a1dfea2f3c74 | |
parent | a441db01b527431a3d7ba169f1afd9c3bc04748f (diff) | |
parent | a47da4b6fecaba81dafe777b25be24e04a428e37 (diff) | |
download | bitcoin-4c69d683f22ad8bc310c901b50b8892da7a3c70c.tar.xz |
Merge #9553: Use z = std::max(x - y, 0) instead of z = x - y; if (z < 0) z = 0;
a47da4b Use z = std::max(x - y, 0); instead of z = x - y; if (z < 0) z = 0; (practicalswift)
-rw-r--r-- | src/addrman.cpp | 6 | ||||
-rw-r--r-- | src/qt/coincontroldialog.cpp | 4 |
2 files changed, 2 insertions, 8 deletions
diff --git a/src/addrman.cpp b/src/addrman.cpp index f3c238fbb2..b6ab4c6305 100644 --- a/src/addrman.cpp +++ b/src/addrman.cpp @@ -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) diff --git a/src/qt/coincontroldialog.cpp b/src/qt/coincontroldialog.cpp index f8aba70d92..d4fd8bd372 100644 --- a/src/qt/coincontroldialog.cpp +++ b/src/qt/coincontroldialog.cpp @@ -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 |