commit 201a10157c0add72a2aac484da3d82220d545714
author Ed Tanous <ed@tanous.net> Wed Apr 03 18:07:28 2024 -0700
committer Ed Tanous <ed@tanous.net> Wed Apr 03 18:28:14 2024 -0700
tree ab08cb3a45a87d0d4fb073ac5c1f84871d7b5aa2
parent a327923d4ba7b47b8ff07fe42ccb025fd9c894b9

Add modernize-redundant-void-arg

Enable this check and fix the failures.

Change-Id: I89b13daf1161be40564367562bb9c8c0c459e1d0
Signed-off-by: Ed Tanous <ed@tanous.net>

src/ExitAirTempSensor.cpp

diff --git a/src/ExitAirTempSensor.cpp b/src/ExitAirTempSensor.cpp
index 59180b5..f0c5c59 100644
--- a/src/ExitAirTempSensor.cpp
+++ b/src/ExitAirTempSensor.cpp
@@ -286,7 +286,7 @@
     objServer.remove_interface(pwmLimitIface);
 }
 
-void CFMSensor::createMaxCFMIface(void)
+void CFMSensor::createMaxCFMIface()
 {
     cfmLimitIface->register_property("Limit", c2 * maxCFM * tachs.size());
     cfmLimitIface->initialize();
@@ -318,12 +318,12 @@
         "xyz.openbmc_project.Sensor.Value");
 }
 
-void CFMSensor::checkThresholds(void)
+void CFMSensor::checkThresholds()
 {
     thresholds::checkThresholds(this);
 }
 
-void CFMSensor::updateReading(void)
+void CFMSensor::updateReading()
 {
     double val = 0.0;
     if (calculate(val))
@@ -524,7 +524,7 @@
     objServer.remove_interface(association);
 }
 
-void ExitAirTempSensor::setupMatches(void)
+void ExitAirTempSensor::setupMatches()
 {
     constexpr const auto matchTypes{
         std::to_array<const char*>({"power", inletTemperatureSensor})};
@@ -632,7 +632,7 @@
         std::array<const char*, 1>{sensorValueInterface});
 }
 
-void ExitAirTempSensor::updateReading(void)
+void ExitAirTempSensor::updateReading()
 {
     double val = 0.0;
     if (calculate(val))
@@ -646,7 +646,7 @@
     }
 }
 
-double ExitAirTempSensor::getTotalCFM(void)
+double ExitAirTempSensor::getTotalCFM()
 {
     double sum = 0;
     for (auto& sensor : cfmSensors)
@@ -825,7 +825,7 @@
     return true;
 }
 
-void ExitAirTempSensor::checkThresholds(void)
+void ExitAirTempSensor::checkThresholds()
 {
     thresholds::checkThresholds(this);
 }