BenchmarkRunner: use std::lround() to round double to int

From clang-tidy bugprone-incorrect-roundings check: > casting (double + 0.5) to integer leads to incorrect rounding; consider using lround (#include <cmath>) instead

BenchmarkRunner: use std::lround() to round double to int
cc7f50e1 · Roman Lebedev · 74e112ae · cc7f50e1
Unverified Commit cc7f50e1 authored Nov 23, 2019 by Roman Lebedev
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

benchmark_runner.cc src/benchmark_runner.cc +1 -1

No files found.
--- a/src/benchmark_runner.cc
+++ b/src/benchmark_runner.cc
@@ -264,7 +264,7 @@ class BenchmarkRunner {

    // So what seems to be the sufficiently-large iteration count? Round up.
    const IterationCount max_next_iters =
-        0.5 + std::max(multiplier * i.iters, i.iters + 1.0);
+        std::lround(std::max(multiplier * i.iters, i.iters + 1.0));
    // But we do have *some* sanity limits though..
    const IterationCount next_iters = std::min(max_next_iters, kMaxIterations);