summaryrefslogtreecommitdiff
path: root/include/openbmc_dbus_rest.hpp
diff options
context:
space:
mode:
authorEd Tanous <edtanous@google.com>2023-02-16 21:28:44 +0300
committerEd Tanous <ed@tanous.net>2023-02-24 20:21:14 +0300
commit26ccae32112679c4653c1e3f8a1203c828bea05c (patch)
treea87d5056294e163b9ddeed9e867bcfb0750a8bb5 /include/openbmc_dbus_rest.hpp
parent7da1c58890e82194ff83ca6c8d55b5c327f2444a (diff)
downloadbmcweb-26ccae32112679c4653c1e3f8a1203c828bea05c.tar.xz
Pass string views by value
string_view should always be passed by value; This commit is a sed replace of the code to make all string_views pass by value, per general coding guidelines[1]. [1] https://quuxplusone.github.io/blog/2021/11/09/pass-string-view-by-value/ Tested: Code compiles. Signed-off-by: Ed Tanous <edtanous@google.com> Change-Id: I55b342a29a0fbfce0a4ed9ea63db6014d03b134c
Diffstat (limited to 'include/openbmc_dbus_rest.hpp')
-rw-r--r--include/openbmc_dbus_rest.hpp3
1 files changed, 1 insertions, 2 deletions
diff --git a/include/openbmc_dbus_rest.hpp b/include/openbmc_dbus_rest.hpp
index 9bf968a40d..30351051dd 100644
--- a/include/openbmc_dbus_rest.hpp
+++ b/include/openbmc_dbus_rest.hpp
@@ -108,8 +108,7 @@ inline bool validateFilename(const std::string& filename)
inline void setErrorResponse(crow::Response& res,
boost::beast::http::status result,
- const std::string& desc,
- const std::string_view msg)
+ const std::string& desc, std::string_view msg)
{
res.result(result);
res.jsonValue["data"]["description"] = desc;