diff options
| author | Ed Tanous <edtanous@google.com> | 2022-07-01 02:50:15 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2022-07-12 21:38:38 +0300 |
| commit | 02cad96e3a83b0c9a01dbc87219c1b8b6681fa67 (patch) | |
| tree | a2dab0f07fe50ed5708f3c6fea43b7d2af5419e7 /include/openbmc_dbus_rest.hpp | |
| parent | bb60f4ded18153718709e23dc1d648eb2ae13743 (diff) | |
| download | bmcweb-02cad96e3a83b0c9a01dbc87219c1b8b6681fa67.tar.xz | |
Fix const correctness issues
cppcheck correctly notes that a lot of variables in the new code can be
const. Make most of them const.
Tested: WIP
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I8f37b6353fd707923f533e1d61c5b5419282bf23
Diffstat (limited to 'include/openbmc_dbus_rest.hpp')
| -rw-r--r-- | include/openbmc_dbus_rest.hpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/openbmc_dbus_rest.hpp b/include/openbmc_dbus_rest.hpp index 0e2af700f6..d0bc29c3d1 100644 --- a/include/openbmc_dbus_rest.hpp +++ b/include/openbmc_dbus_rest.hpp @@ -2340,7 +2340,7 @@ inline void nlohmann::json& propertyItem = propertiesObj[name]; crow::connections::systemBus->async_send( m, [&propertyItem, - asyncResp](boost::system::error_code& e, + asyncResp](const boost::system::error_code& e, sdbusplus::message::message& msg) { if (e) { @@ -2420,7 +2420,7 @@ inline void requestRoutes(App& app) std::sort(names.begin(), names.end()); asyncResp->res.jsonValue["status"] = "ok"; auto& objectsSub = asyncResp->res.jsonValue["objects"]; - for (auto& name : names) + for (const auto& name : names) { nlohmann::json::object_t object; object["name"] = name; |