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 /redfish-core/lib/task.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 'redfish-core/lib/task.hpp')
-rw-r--r-- | redfish-core/lib/task.hpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/redfish-core/lib/task.hpp b/redfish-core/lib/task.hpp index dcad059655..88926146f0 100644 --- a/redfish-core/lib/task.hpp +++ b/redfish-core/lib/task.hpp @@ -120,7 +120,7 @@ struct TaskData : std::enable_shared_from_this<TaskData> if (tasks.size() >= maxTaskCount) { - auto& last = tasks.front(); + const auto& last = tasks.front(); // destroy all references last->timer.cancel(); @@ -389,7 +389,7 @@ inline void requestRoutesTask(App& app) return; } - std::shared_ptr<task::TaskData>& ptr = *find; + const std::shared_ptr<task::TaskData>& ptr = *find; asyncResp->res.jsonValue["@odata.type"] = "#Task.v1_4_3.Task"; asyncResp->res.jsonValue["Id"] = strParam; |