From b025a007b924cc16da84c07a5d35118695cb9bcc Mon Sep 17 00:00:00 2001 From: Scott Hutchinson Date: Sat, 23 Jan 2021 12:58:20 -0800 Subject: [PATCH] Wrap all std::min and std::max calls in parentheses --- include/internal/benchmark/detail/catch_estimate_clock.hpp | 4 +++- include/internal/benchmark/detail/catch_stats.cpp | 4 ++-- include/internal/benchmark/detail/catch_stats.hpp | 4 ++-- 3 files changed, 7 insertions(+), 5 deletions(-) diff --git a/include/internal/benchmark/detail/catch_estimate_clock.hpp b/include/internal/benchmark/detail/catch_estimate_clock.hpp index 055c582500..391f5d24ce 100644 --- a/include/internal/benchmark/detail/catch_estimate_clock.hpp +++ b/include/internal/benchmark/detail/catch_estimate_clock.hpp @@ -68,7 +68,9 @@ namespace Catch { } template EnvironmentEstimate> estimate_clock_cost(FloatDuration resolution) { - auto time_limit = std::min(resolution * clock_cost_estimation_tick_limit, FloatDuration(clock_cost_estimation_time_limit)); + auto time_limit = (std::min)( + resolution * clock_cost_estimation_tick_limit, + FloatDuration(clock_cost_estimation_time_limit)); auto time_clock = [](int k) { return Detail::measure([k] { for (int i = 0; i < k; ++i) { diff --git a/include/internal/benchmark/detail/catch_stats.cpp b/include/internal/benchmark/detail/catch_stats.cpp index b85b740ea3..dcb51b15ea 100644 --- a/include/internal/benchmark/detail/catch_stats.cpp +++ b/include/internal/benchmark/detail/catch_stats.cpp @@ -152,7 +152,7 @@ namespace Catch { double sb = stddev.point; double mn = mean.point / n; double mg_min = mn / 2.; - double sg = std::min(mg_min / 4., sb / std::sqrt(n)); + double sg = (std::min)(mg_min / 4., sb / std::sqrt(n)); double sg2 = sg * sg; double sb2 = sb * sb; @@ -171,7 +171,7 @@ namespace Catch { return (nc / n) * (sb2 - nc * sg2); }; - return std::min(var_out(1), var_out(std::min(c_max(0.), c_max(mg_min)))) / sb2; + return (std::min)(var_out(1), var_out((std::min)(c_max(0.), c_max(mg_min)))) / sb2; } diff --git a/include/internal/benchmark/detail/catch_stats.hpp b/include/internal/benchmark/detail/catch_stats.hpp index 71a460d900..1bcbc259b9 100644 --- a/include/internal/benchmark/detail/catch_stats.hpp +++ b/include/internal/benchmark/detail/catch_stats.hpp @@ -138,8 +138,8 @@ namespace Catch { double b2 = bias - z1; double a1 = a(b1); double a2 = a(b2); - auto lo = std::max(cumn(a1), 0); - auto hi = std::min(cumn(a2), n - 1); + auto lo = (std::max)(cumn(a1), 0); + auto hi = (std::min)(cumn(a2), n - 1); return { point, resample[lo], resample[hi], confidence_level }; }