cleanup: move from copy and pointer to just const reference
Originally code passed the object by value, and then later by pointer to
that copy. Convert the code to be more performant by using a const
reference at all layers of access for this object.
Change-Id: Icdf0dfdb54d8adc29af4d05d841533626a484921
Signed-off-by: Patrick Venture <venture@google.com>
diff --git a/pid/fancontroller.cpp b/pid/fancontroller.cpp
index f085927..f622698 100644
--- a/pid/fancontroller.cpp
+++ b/pid/fancontroller.cpp
@@ -25,7 +25,7 @@
std::unique_ptr<PIDController>
FanController::CreateFanPid(ZoneInterface* owner, const std::string& id,
const std::vector<std::string>& inputs,
- ec::pidinfo initial)
+ const ec::pidinfo& initial)
{
if (inputs.size() == 0)
{
@@ -34,7 +34,7 @@
auto fan = std::make_unique<FanController>(id, inputs, owner);
ec::pid_info_t* info = fan->get_pid_info();
- InitializePIDStruct(info, &initial);
+ InitializePIDStruct(info, initial);
return fan;
}