control/zone: Reference instead of copying timer data
Tested:
Built and ran through unit tests.
Change-Id: I9a98d9a3eaee47885b18b642ffd01a38a864dd02
Signed-off-by: William A. Kennington III <wak@google.com>
diff --git a/control/zone.cpp b/control/zone.cpp
index ec0f944..1ca0465 100644
--- a/control/zone.cpp
+++ b/control/zone.cpp
@@ -441,7 +441,10 @@
);
auto timer = std::make_unique<util::Timer>(
_eventLoop,
- std::bind(&Zone::timerExpired, this, group, actions)
+ std::bind(&Zone::timerExpired,
+ this,
+ std::cref(std::get<Group>(*data)),
+ std::cref(std::get<std::vector<Action>>(*data)))
);
if (!timer->running())
{
@@ -451,7 +454,8 @@
_timerEvents.emplace_back(std::move(data), std::move(timer));
}
-void Zone::timerExpired(Group eventGroup, std::vector<Action> eventActions)
+void Zone::timerExpired(const Group& eventGroup,
+ const std::vector<Action>& eventActions)
{
// Perform the actions
std::for_each(eventActions.begin(),