commit 695888de8bfbee0f336ad1bd7c517427e601e62a
author Pavithra Barithaya <pavithrabarithaya07@gmail.com> Fri Jun 21 12:01:30 2024 -0500
committer Pavithra Barithaya <pavithrabarithaya07@gmail.com> Fri Jul 05 15:04:03 2024 +0530
tree e6c1c1c0d87c4a5739e5e61397a63b75664b1171
parent 319eda484d41d1a30b11f3da3370371450ecd759

clang-tidy: Enable readability-redundant-string-cstr check

This check finds unnecessary calls to std::string::c_str() and
std::string::data().

Change-Id: I43e747b95eb042650668da7dd57a8d4b8463426a
Signed-off-by: Pavithra Barithaya <pavithrabarithaya07@gmail.com>

utils.cpp

diff --git a/utils.cpp b/utils.cpp
index 0df843b..8ebe270 100644
--- a/utils.cpp
+++ b/utils.cpp
@@ -242,7 +242,7 @@
     auto bmcPath = sdbusplus::message::object_path(BMC::namespace_path::value) /
                    BMC::namespace_path::bmc;
 
-    auto bmcState = getProperty(bus, bmcPath.str.c_str(), BMC::interface,
+    auto bmcState = getProperty(bus, bmcPath.str, BMC::interface,
                                 "CurrentBMCState");
 
     if (sdbusplus::message::convert_from_string<BMC::BMCState>(bmcState) !=