Implement clang-tidy fixes
clang-tidy allows the CI robot to check many things via static analysis.
All changes here were made by the clang-tidy robot, and include a number
of modernization fixes. updating the tidy file will be done at a later
date.
Signed-off-by: Ed Tanous <etanous@nvidia.com>
Change-Id: I98cc4d600a3c589675507958f6d2350b2141216b
diff --git a/pid/pidloop.cpp b/pid/pidloop.cpp
index 888419b..9d30798 100644
--- a/pid/pidloop.cpp
+++ b/pid/pidloop.cpp
@@ -32,7 +32,7 @@
namespace pid_control
{
-static void processThermals(std::shared_ptr<ZoneInterface> zone)
+static void processThermals(const std::shared_ptr<ZoneInterface>& zone)
{
// Get the latest margins.
zone->updateSensors();
@@ -45,12 +45,14 @@
zone->determineMaxSetPointRequest();
}
-void pidControlLoop(std::shared_ptr<ZoneInterface> zone,
- std::shared_ptr<boost::asio::steady_timer> timer,
+void pidControlLoop(const std::shared_ptr<ZoneInterface>& zone,
+ const std::shared_ptr<boost::asio::steady_timer>& timer,
const bool* isCanceling, bool first, uint64_t cycleCnt)
{
if (*isCanceling)
+ {
return;
+ }
std::chrono::steady_clock::time_point nextTime;