From af37f0180a4f0823f2c0e51adc5a7609d09c8bdc Mon Sep 17 00:00:00 2001 From: Arnaud Giersch Date: Tue, 24 Nov 2020 09:19:32 +0100 Subject: [PATCH] Use a lambda for comparator. --- .../replay_multiple_manual.cpp | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/examples/smpi/replay_multiple_manual_deploy/replay_multiple_manual.cpp b/examples/smpi/replay_multiple_manual_deploy/replay_multiple_manual.cpp index c82a7c7e5a..1d8a49532d 100644 --- a/examples/smpi/replay_multiple_manual_deploy/replay_multiple_manual.cpp +++ b/examples/smpi/replay_multiple_manual_deploy/replay_multiple_manual.cpp @@ -48,13 +48,6 @@ struct Job { static std::vector hosts; static int noise_between_jobs; -static bool job_comparator(const std::unique_ptr& j1, const std::unique_ptr& j2) -{ - if (j1->starting_time == j2->starting_time) - return j1->smpi_app_name < j2->smpi_app_name; - return j1->starting_time < j2->starting_time; -} - static void smpi_replay_process(Job* job, simgrid::s4u::BarrierPtr barrier, int rank) { XBT_INFO("Replaying rank %d of job %d (smpi_app '%s')", rank, job->unique_job_number, job->smpi_app_name.c_str()); @@ -197,8 +190,11 @@ static std::vector> all_jobs(const std::string& workload_fi // Jobs are sorted by ascending date, then by lexicographical order of their // application names - sort(jobs.begin(), jobs.end(), job_comparator); - + sort(jobs.begin(), jobs.end(), [](auto const& j1, auto const& j2) { + if (j1->starting_time == j2->starting_time) + return j1->smpi_app_name < j2->smpi_app_name; + return j1->starting_time < j2->starting_time; + }); for (unsigned int i = 0; i < jobs.size(); ++i) jobs[i]->unique_job_number = i; -- 2.20.1