diff --git a/src/app/application.cpp b/src/app/application.cpp index 56788f5a7..2074f1e85 100644 --- a/src/app/application.cpp +++ b/src/app/application.cpp @@ -427,12 +427,12 @@ void Application::setFileLoggerDeleteOld(const bool value) int Application::fileLoggerMaxSize() const { const int val = m_storeFileLoggerMaxSize.get(DEFAULT_FILELOG_SIZE); - return std::min(std::max(val, MIN_FILELOG_SIZE), MAX_FILELOG_SIZE); + return std::clamp(val, MIN_FILELOG_SIZE, MAX_FILELOG_SIZE); } void Application::setFileLoggerMaxSize(const int bytes) { - const int clampedValue = std::min(std::max(bytes, MIN_FILELOG_SIZE), MAX_FILELOG_SIZE); + const int clampedValue = std::clamp(bytes, MIN_FILELOG_SIZE, MAX_FILELOG_SIZE); if (m_fileLogger) m_fileLogger->setMaxSize(clampedValue); m_storeFileLoggerMaxSize = clampedValue; @@ -441,12 +441,12 @@ void Application::setFileLoggerMaxSize(const int bytes) int Application::fileLoggerAge() const { const int val = m_storeFileLoggerAge.get(1); - return std::min(std::max(val, 1), 365); + return std::clamp(val, 1, 365); } void Application::setFileLoggerAge(const int value) { - m_storeFileLoggerAge = std::min(std::max(value, 1), 365); + m_storeFileLoggerAge = std::clamp(value, 1, 365); } int Application::fileLoggerAgeType() const diff --git a/src/base/bittorrent/sessionimpl.cpp b/src/base/bittorrent/sessionimpl.cpp index aa67d6d61..7ef1d22af 100644 --- a/src/base/bittorrent/sessionimpl.cpp +++ b/src/base/bittorrent/sessionimpl.cpp @@ -266,11 +266,7 @@ namespace { return [lower, upper](const T value) -> T { - if (value < lower) - return lower; - if (value > upper) - return upper; - return value; + return std::clamp(value, lower, upper); }; } diff --git a/src/base/preferences.cpp b/src/base/preferences.cpp index c4ab472c6..5556c8add 100644 --- a/src/base/preferences.cpp +++ b/src/base/preferences.cpp @@ -29,6 +29,7 @@ #include "preferences.h" +#include #include #ifdef Q_OS_MACOS @@ -2122,7 +2123,7 @@ int Preferences::addNewTorrentDialogSavePathHistoryLength() const void Preferences::setAddNewTorrentDialogSavePathHistoryLength(const int value) { - const int clampedValue = qBound(0, value, 99); + const int clampedValue = std::clamp(value, 0, 99); const int oldValue = addNewTorrentDialogSavePathHistoryLength(); if (clampedValue == oldValue) return;