diff options
| author | Ed Tanous <edtanous@google.com> | 2022-06-06 19:57:26 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2022-06-30 21:48:06 +0300 |
| commit | 4e23a444e8503fd03ab40de2844c783e142ebef8 (patch) | |
| tree | f3d9d1fbe5dde9f0d5e1bc23bfb2ac326bf23a4f /redfish-core/lib/task.hpp | |
| parent | 69320d54efe41fa5b0947adab5702d9023cc1ee3 (diff) | |
| download | bmcweb-4e23a444e8503fd03ab40de2844c783e142ebef8.tar.xz | |
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 <edtanous@google.com>
Change-Id: Ia428463313b075c69614fdb326e8c5c094e7adde
Diffstat (limited to 'redfish-core/lib/task.hpp')
| -rw-r--r-- | redfish-core/lib/task.hpp | 2 |
1 files changed, 1 insertions, 1 deletions
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()) |