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 /http/utility.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 'http/utility.hpp')
-rw-r--r-- | http/utility.hpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/http/utility.hpp b/http/utility.hpp index 0b1743f228..7b0af130a6 100644 --- a/http/utility.hpp +++ b/http/utility.hpp @@ -655,7 +655,8 @@ class UrlSegmentMatcherVisitor return std::string_view(segment.data(), segment.size()) == expected; } - UrlSegmentMatcherVisitor(const boost::urls::string_value& segmentIn) : + explicit UrlSegmentMatcherVisitor( + const boost::urls::string_value& segmentIn) : segment(segmentIn) {} |