commit e6b2be523a942ae753a9f36a9f5c6bc80b24ca79
author George Liu <liuxiwei@ieisystem.com> Tue Aug 19 13:57:43 2025 +0800
committer George Liu <liuxiwei@ieisystem.com> Thu Sep 25 15:21:59 2025 +0800
tree dedfbb601680bfd3dd7c99fbbb35b1314afc5a47
parent c7a4da5ebd9de338595e8a31740eb04b1c57a69b

replace pointer parameters with references

The methods declared in sensorhandler.hpp are all defined with
pointer parameters. Since these parameters should always be valid and
not null, using raw pointers is unnecessary and can be misleading, so
passing them as references makes the API safer and more idiomatic in
modern C++.

Change-Id: Iaec6caeb1e04fcc38cc244dd66634724c62dd0c1
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

sensordatahandler.hpp

diff --git a/sensordatahandler.hpp b/sensordatahandler.hpp
index 3d56971..c5dbcfc 100644
--- a/sensordatahandler.hpp
+++ b/sensordatahandler.hpp
@@ -175,7 +175,7 @@
     sdbusplus::bus_t bus{ipmid_get_sd_bus_connection()};
     GetSensorResponse response{};
 
-    enableScanning(&response);
+    enableScanning(response);
     try
     {
         auto service = ipmi::getService(bus, sensorInfo.sensorInterface,
@@ -186,7 +186,7 @@
             sensorInfo.propertyInterfaces.begin()->second.begin()->first);
 
         setAssertionBytes(static_cast<uint16_t>(std::get<T>(propValue)),
-                          &response);
+                          response);
     }
     catch (const std::exception& e)
     {
@@ -214,7 +214,7 @@
 
     GetSensorResponse response{};
 
-    enableScanning(&response);
+    enableScanning(response);
 
     auto service = ipmi::getService(bus, sensorInfo.sensorInterface,
                                     sensorInfo.sensorPath);
@@ -284,7 +284,7 @@
         maxClamp = std::numeric_limits<uint8_t>::max();
     }
     setReading(static_cast<uint8_t>(std::clamp(rawData, minClamp, maxClamp)),
-               &response);
+               response);
 
     if (!std::isfinite(value))
     {
@@ -391,7 +391,7 @@
     uint8_t id, const Info& sensorInfo, const PropertyMap& properties)
 {
     GetSensorResponse response{};
-    enableScanning(&response);
+    enableScanning(response);
 
     auto iter = properties.find(
         sensorInfo.propertyInterfaces.begin()->second.begin()->first);
@@ -401,7 +401,7 @@
     }
 
     setAssertionBytes(static_cast<uint16_t>(std::get<T>(iter->second)),
-                      &response);
+                      response);
 
     if (!sensorCacheMap[id].has_value())
     {
@@ -446,7 +446,7 @@
 
     GetSensorResponse response{};
 
-    enableScanning(&response);
+    enableScanning(response);
 
     iter = properties.find(
         sensorInfo.propertyInterfaces.begin()->second.begin()->first);
@@ -471,7 +471,7 @@
             lg2::error("Value out of range");
             throw std::out_of_range("Value out of range");
         }
-        setReading(static_cast<int8_t>(rawData), &response);
+        setReading(static_cast<int8_t>(rawData), response);
     }
     else
     {
@@ -481,7 +481,7 @@
             lg2::error("Value out of range");
             throw std::out_of_range("Value out of range");
         }
-        setReading(static_cast<uint8_t>(rawData), &response);
+        setReading(static_cast<uint8_t>(rawData), response);
     }
 
     if (!std::isfinite(value))