From 9bab26fb105752005745f0beacd36b3e5a2f6053 Mon Sep 17 00:00:00 2001 From: Jure Bajic Date: Mon, 5 Dec 2022 15:50:40 +0100 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Kostas Kyrimis --- tests/benchmark/data_structures_remove.cpp | 2 +- tests/benchmark/skip_list_common.hpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/benchmark/data_structures_remove.cpp b/tests/benchmark/data_structures_remove.cpp index 0d7baaccf..641ad9453 100644 --- a/tests/benchmark/data_structures_remove.cpp +++ b/tests/benchmark/data_structures_remove.cpp @@ -41,7 +41,7 @@ static void BM_BenchmarkRemoveSkipList(::benchmark::State &state) { utils::SkipList skip_list; PrepareData(skip_list, state.range(0)); - // So we can also have elements that does don't exist + // So we can also have elements that don't exist std::mt19937 i_generator(std::random_device{}()); std::uniform_int_distribution i_distribution(0, state.range(0) * 2); int64_t removed_elems{0}; diff --git a/tests/benchmark/skip_list_common.hpp b/tests/benchmark/skip_list_common.hpp index 8dd4c705b..54e62ed7e 100644 --- a/tests/benchmark/skip_list_common.hpp +++ b/tests/benchmark/skip_list_common.hpp @@ -99,7 +99,7 @@ inline void RunTest(std::function &, Stats &)> test std::cout << " Successful find: " << stats.succ[3] << std::endl; std::cout << std::endl; - const auto tot = stats.succ[0] + stats.succ[1] + stats.succ[2] + stats.succ[3]; + const auto tot = std::accumulate(stats.succ.begin(), + stats.succ.begin() + 3, 0); const auto tops = stats.total; std::cout << "Total successful: " << tot << " (" << tot / FLAGS_duration << " calls/s)" << std::endl;