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/include/cors_preflight.hpp b/include/cors_preflight.hpp
index 1cd8564..29d7475 100644
--- a/include/cors_preflight.hpp
+++ b/include/cors_preflight.hpp
@@ -10,8 +10,8 @@
 {
     BMCWEB_ROUTE(app, "<str>")
         .methods(boost::beast::http::verb::options)(
-            [](const crow::Request&, const std::shared_ptr<bmcweb::AsyncResp>&,
-               const std::string&) {
+            [](const crow::Request& /*req*/,
+               const std::shared_ptr<bmcweb::AsyncResp>&, const std::string&) {
                 // An empty body handler that simply returns the headers bmcweb
                 // uses This allows browsers to do their CORS preflight checks
             });