commit d99148b3a0e72775ec217ff4060e3021c50299c3
author Patrick Venture <venture@google.com> Sat Oct 13 10:06:13 2018 -0700
committer Patrick Venture <venture@google.com> Wed Oct 17 22:45:46 2018 +0000
tree 56cd6ea977fdc7877d3beaf2c11b4a9e97898b0b
parent 4d1a5dd4a55b8886ce8fc7512578ba3438faf386

cleanup: c-style casting

[ipmisensor.cpp:308]:    (style) C-style pointer casting
[sensorhandler.cpp:302]: (style) C-style pointer casting
[sensorhandler.cpp:376]: (style) C-style pointer casting
[sensorhandler.cpp:404]: (style) C-style pointer casting
[sensorhandler.cpp:405]: (style) C-style pointer casting
[storageaddsel.cpp:157]: (style) C-style pointer casting
[storageaddsel.cpp:187]: (style) C-style pointer casting
Added missing const in reportSensorEventAssert and
reportSensorEventDeassert as the *pTable->func(...) take the const
pointer and the casting then was using the proper const casting.

Change-Id: I54a591b4e5e678b8ec3cae2633617a5f7aac7a66
Signed-off-by: Patrick Venture <venture@google.com>

sensorhandler.cpp

diff --git a/sensorhandler.cpp b/sensorhandler.cpp
index eafb5d0..bb06b94 100644
--- a/sensorhandler.cpp
+++ b/sensorhandler.cpp
@@ -299,7 +299,7 @@
                                     ipmi_data_len_t data_len,
                                     ipmi_context_t context)
 {
-    sensor_data_t* reqptr = (sensor_data_t*)request;
+    auto reqptr = static_cast<sensor_data_t*>(request);
     ipmi_ret_t rc = IPMI_CC_OK;
 
     printf("IPMI GET_SENSOR_TYPE [0x%02X]\n", reqptr->sennum);
@@ -373,7 +373,7 @@
                                ipmi_request_t request, ipmi_response_t response,
                                ipmi_data_len_t data_len, ipmi_context_t context)
 {
-    sensor_data_t* reqptr = (sensor_data_t*)request;
+    auto reqptr = static_cast<sensor_data_t*>(request);
 
     log<level::DEBUG>("IPMI SET_SENSOR",
                       entry("SENSOR_NUM=0x%02x", reqptr->sennum));
@@ -401,8 +401,8 @@
                                        ipmi_data_len_t data_len,
                                        ipmi_context_t context)
 {
-    sensor_data_t* reqptr = (sensor_data_t*)request;
-    sensorreadingresp_t* resp = (sensorreadingresp_t*)response;
+    auto reqptr = static_cast<sensor_data_t*>(request);
+    auto resp = static_cast<sensorreadingresp_t*>(response);
     ipmi::sensor::GetSensorResponse getResponse{};
     static constexpr auto scanningEnabledBit = 6;