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
diff --git a/http/http_client.hpp b/http/http_client.hpp
index 63c2611..f59438b 100644
--- a/http/http_client.hpp
+++ b/http/http_client.hpp
@@ -632,7 +632,7 @@
// Used as a dummy callback by sendData() in order to call
// sendDataWithCallback()
- static void genericResHandler(Response& res)
+ static void genericResHandler(const Response& res)
{
BMCWEB_LOG_DEBUG << "Response handled with return code: "
<< std::to_string(res.resultInt());
@@ -660,7 +660,7 @@
const boost::beast::http::verb verb,
const std::string& retryPolicyName)
{
- std::function<void(Response&)> cb = genericResHandler;
+ const std::function<void(const Response&)> cb = genericResHandler;
sendDataWithCallback(data, id, destIP, destPort, destUri, httpHeader,
verb, retryPolicyName, cb);
}