From 4e23a444e8503fd03ab40de2844c783e142ebef8 Mon Sep 17 00:00:00 2001 From: Ed Tanous Date: Mon, 6 Jun 2022 09:57:26 -0700 Subject: Require explicit decorator on one arg constructors We essentially follow this rule already, not relying on implicit operators, although there are a number of cases where in theory we could've implicitly constructed an object. This commit enables the clang-tidy check. Tested: Code compiles, passes clang-tidy. Signed-off-by: Ed Tanous Change-Id: Ia428463313b075c69614fdb326e8c5c094e7adde --- redfish-core/lib/task.hpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'redfish-core/lib/task.hpp') diff --git a/redfish-core/lib/task.hpp b/redfish-core/lib/task.hpp index b1b1d22fb3..dcad059655 100644 --- a/redfish-core/lib/task.hpp +++ b/redfish-core/lib/task.hpp @@ -39,7 +39,7 @@ constexpr bool completed = true; struct Payload { - Payload(const crow::Request& req) : + explicit Payload(const crow::Request& req) : targetUri(req.url), httpOperation(req.methodString()), httpHeaders(nlohmann::json::array()) -- cgit v1.2.3