From 242bab5395d808b0c25b2df8971c11a5f8932001 Mon Sep 17 00:00:00 2001 From: Charles Kerr Date: Sun, 5 Apr 2015 18:27:08 -0500 Subject: in SimpleAlarmQueue, use references instead of copies when copies aren't necessary --- src/alarm-queue-simple.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/alarm-queue-simple.cpp') diff --git a/src/alarm-queue-simple.cpp b/src/alarm-queue-simple.cpp index db0fd21..ae7722c 100644 --- a/src/alarm-queue-simple.cpp +++ b/src/alarm-queue-simple.cpp @@ -110,14 +110,14 @@ private: const auto now = m_clock->localtime(); const auto beginning_of_minute = now.start_of_minute(); - const auto appointments = m_planner->appointments().get(); + const auto& appointments = m_planner->appointments().get(); g_debug ("planner has %zu appointments in it", (size_t)appointments.size()); for(const auto& appointment : appointments) { for(const auto& alarm : appointment.alarms) { - const std::pair trig{appointment.uid, alarm.time}; + const std::pair trig{appointment.uid, alarm.time}; if (m_triggered.count(trig)) continue; @@ -145,7 +145,7 @@ private: for (const auto& alarm : appointment.alarms) { - const auto trig = std::make_pair(appointment.uid, alarm.time); + const std::pair trig{appointment.uid, alarm.time}; if (m_triggered.count(trig)) // did we already use this one? continue; -- cgit v1.2.3