Enable readability-named-parameter checks
We don't have too many violations here, probably because we don't have
many optional parameters. Fix the existing instances, and enable the
check.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I4d512f0ec90b060fb60a42fe3cd6ba72fb6c6bcb
diff --git a/redfish-core/lib/message_registries.hpp b/redfish-core/lib/message_registries.hpp
index cfa2bf5..c2848c3 100644
--- a/redfish-core/lib/message_registries.hpp
+++ b/redfish-core/lib/message_registries.hpp
@@ -28,7 +28,8 @@
{
inline void handleMessageRegistryFileCollectionGet(
- const crow::Request&, const std::shared_ptr<bmcweb::AsyncResp>& asyncResp)
+ const crow::Request& /*req*/,
+ const std::shared_ptr<bmcweb::AsyncResp>& asyncResp)
{
// Collections don't include the static data added by SubRoute
// because it has a duplicate entry for members
@@ -59,7 +60,8 @@
}
inline void handleMessageRoutesMessageRegistryFileGet(
- const crow::Request&, const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
+ const crow::Request& /*req*/,
+ const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
const std::string& registry)
{
const message_registries::Header* header = nullptr;
@@ -123,7 +125,8 @@
}
inline void handleMessageRegistryGet(
- const crow::Request&, const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
+ const crow::Request& /*req*/,
+ const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
const std::string& registry, const std::string& registryMatch)
{