summaryrefslogtreecommitdiffstats
path: root/src/rgw/rgw_frontend.h
diff options
context:
space:
mode:
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>2018-04-25 09:30:56 +0200
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>2018-05-02 17:01:58 +0200
commit39ffec281af577c766c619d568f7a7051d108b00 (patch)
tree79c527cc3871b9b113f1aabffb3cef9647ebb7be /src/rgw/rgw_frontend.h
parentMerge PR #21762 into master (diff)
downloadceph-39ffec281af577c766c619d568f7a7051d108b00.tar.xz
ceph-39ffec281af577c766c619d568f7a7051d108b00.zip
misc: mark constructors as explicit
Set 218 constructors as explicit to avoid implicit usage. Fix for cppcheck warning: Class has a constructor with 1 argument that is not explicit. Such constructors should in general be explicit for type safety reasons. Using the explicit keyword in the constructor means some mistakes when using the class can be avoided. For more information check: https://www.codeproject.com/Articles/28663/Explicit-Constructor-in-C Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Diffstat (limited to 'src/rgw/rgw_frontend.h')
-rw-r--r--src/rgw/rgw_frontend.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/rgw/rgw_frontend.h b/src/rgw/rgw_frontend.h
index c5d9613536d..75bdd86d748 100644
--- a/src/rgw/rgw_frontend.h
+++ b/src/rgw/rgw_frontend.h
@@ -29,7 +29,7 @@ class RGWFrontendConfig {
std::multimap<std::string, std::string>& config_map);
public:
- RGWFrontendConfig(const std::string& config)
+ explicit RGWFrontendConfig(const std::string& config)
: config(config) {
}
@@ -85,7 +85,7 @@ struct RGWMongooseEnv : public RGWProcessEnv {
static constexpr bool prioritize_write = true;
RWLock mutex;
- RGWMongooseEnv(const RGWProcessEnv &env)
+ explicit RGWMongooseEnv(const RGWProcessEnv &env)
: RGWProcessEnv(env),
mutex("RGWCivetWebFrontend", false, true, prioritize_write) {
}