diff options
author | Ed Tanous <edtanous@google.com> | 2023-02-16 21:28:44 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2023-02-24 20:21:14 +0300 |
commit | 26ccae32112679c4653c1e3f8a1203c828bea05c (patch) | |
tree | a87d5056294e163b9ddeed9e867bcfb0750a8bb5 /include/openbmc_dbus_rest.hpp | |
parent | 7da1c58890e82194ff83ca6c8d55b5c327f2444a (diff) | |
download | bmcweb-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.hpp | 3 |
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; |