libpldmresponder: platform: Remove awkward reinterpret_cast()
Arrays naturally decay to pointers. The reinterpret_cast() is required
if you're trying to take the address of the array and coerce the type
into the array's underlying type, but if we exploit the natural decay we
can drop the reinterpret_cast():
```
$ g++ -std=c++20 -x c++ -Wall -pedantic -Werror - \
> <<< 'unsigned char a[4]; unsigned char *s = &a; int main() { }'
<stdin>:1:40: error: cannot convert ‘unsigned char (*)[4]’ to ‘unsigned char*’ in initialization
$ g++ -std=c++20 -x c++ -Wall -pedantic -Werror - \
> <<< 'unsigned char a[4]; unsigned char *s = a; int main() { }'
$ echo $?
0
$
```
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I6ab3912d9065e33d2c36ea916c9abaa98a2b3076
diff --git a/libpldmresponder/platform.cpp b/libpldmresponder/platform.cpp
index 31a7d6c..fa6058d 100644
--- a/libpldmresponder/platform.cpp
+++ b/libpldmresponder/platform.cpp
@@ -605,8 +605,7 @@
}
int rc = decode_set_numeric_effecter_value_req(
- request, payloadLength, &effecterId, &effecterDataSize,
- reinterpret_cast<uint8_t*>(&effecterValue));
+ request, payloadLength, &effecterId, &effecterDataSize, effecterValue);
if (rc == PLDM_SUCCESS)
{