Make some single arg constructors explicit

Change-Id: Id5023fef37dd0b84a83a90fb0a3c52b0be31974c

diff --git a/src/manager/client/client-common.h b/src/manager/client/client-common.h
index 032f2f9..d285b5d 100644 (file)
--- a/src/manager/client/client-common.h
+++ b/src/manager/client/client-common.h
@@ -1,5 +1,5 @@
 /*
- *  Copyright (c) 2000 - 2019 Samsung Electronics Co., Ltd All Rights Reserved
+ *  Copyright (c) 2000-2019 Samsung Electronics Co., Ltd. All rights reserved
  *
  *  Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
@@ -53,7 +53,7 @@ namespace CKM {
 
 class AliasSupport {
 public:
-       AliasSupport(const Alias &alias);
+       explicit AliasSupport(const Alias &alias);
 
        const ClientId &getOwner() const;
        const Name &getName() const;
@@ -87,7 +87,7 @@ protected:
 
 class ServiceConnection {
 public:
-       ServiceConnection(const char *service_interface);
+       explicit ServiceConnection(const char *service_interface);
 
        // roundtrip: send and receive
        int processRequest(const CKM::RawBuffer &send_buf,

diff --git a/src/manager/initial-values/BufferHandler.h b/src/manager/initial-values/BufferHandler.h
index 1b45839..b85c492 100644 (file)
--- a/src/manager/initial-values/BufferHandler.h
+++ b/src/manager/initial-values/BufferHandler.h
@@ -1,5 +1,5 @@
 /*
- *  Copyright (c) 2015 - 2018 Samsung Electronics Co., Ltd All Rights Reserved
+ *  Copyright (c) 2015-2019 Samsung Electronics Co., Ltd. All rights reserved
  *
  *  Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
@@ -35,7 +35,7 @@ class BufferHandler : public XML::Parser::ElementHandler {
 public:
        typedef std::shared_ptr<BufferHandler> BufferHandlerPtr;
 
-       BufferHandler(EncodingType type);
+       explicit BufferHandler(EncodingType type);
        virtual ~BufferHandler();
 
        virtual void Start(const XML::Parser::Attributes &);

diff --git a/src/manager/service/db-crypto.h b/src/manager/service/db-crypto.h
index ad0ef12..4642b74 100644 (file)
--- a/src/manager/service/db-crypto.h
+++ b/src/manager/service/db-crypto.h
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2014 Samsung Electronics Co., Ltd All Rights Reserved
+ * Copyright (c) 2014-2019 Samsung Electronics Co., Ltd. All rights reserved
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -133,7 +133,7 @@ public:
 
        class Transaction {
        public:
-               Transaction(Crypto *db) : m_db(db), m_inTransaction(false)
+               explicit Transaction(Crypto *db) : m_db(db), m_inTransaction(false)
                {
                        if (!m_db->m_inUserTransaction) {
                                try {

diff --git a/src/manager/service/encryption-logic.h b/src/manager/service/encryption-logic.h
index b51439e..627ed85 100644 (file)
--- a/src/manager/service/encryption-logic.h
+++ b/src/manager/service/encryption-logic.h
@@ -1,5 +1,5 @@
 /*
- *  Copyright (c) 2000 - 2015 Samsung Electronics Co., Ltd All Rights Reserved
+ *  Copyright (c) 2000-2019 Samsung Electronics Co., Ltd. All rights reserved
  *
  *  Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
@@ -32,7 +32,7 @@ namespace CKM {
 
 class EncryptionLogic {
 public:
-       EncryptionLogic(IEncryptionService &service) : m_service(service) {}
+       explicit EncryptionLogic(IEncryptionService &service) : m_service(service) {}
        virtual ~EncryptionLogic() {}
 
        void Crypt(const CryptoRequest &request);

diff --git a/src/manager/service/file-system.h b/src/manager/service/file-system.h
index 287fbf6..8c77755 100644 (file)
--- a/src/manager/service/file-system.h
+++ b/src/manager/service/file-system.h
@@ -1,5 +1,5 @@
 /*
- *  Copyright (c) 2000 - 2014 Samsung Electronics Co., Ltd All Rights Reserved
+ *  Copyright (c) 2000-2019 Samsung Electronics Co., Ltd. All rights reserved
  *
  *  Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
@@ -32,7 +32,7 @@ typedef std::vector<uid_t> UidVector;
 
 class FileSystem {
 public:
-       FileSystem(uid_t uid);
+       explicit FileSystem(uid_t uid);
 
        std::string getDBPath() const;
 

diff --git a/src/manager/service/key-provider.h b/src/manager/service/key-provider.h
index 668786e..713e952 100644 (file)
--- a/src/manager/service/key-provider.h
+++ b/src/manager/service/key-provider.h
@@ -1,5 +1,5 @@
 /*
- *  Copyright (c) 2014 - 2019 Samsung Electronics Co., Ltd All Rights Reserved
+ *  Copyright (c) 2014-2019 Samsung Electronics Co., Ltd. All rights reserved
  *
  *  Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
@@ -98,7 +98,7 @@ private:
 class KeyAndInfoContainer {
 public:
        KeyAndInfoContainer();
-       KeyAndInfoContainer(const unsigned char *);
+       explicit KeyAndInfoContainer(const unsigned char *);
        KeyAndInfo &getKeyAndInfo();
        void setKeyInfoKeyLength(const unsigned int);
        void setKeyInfo(const KeyComponentsInfo *);

diff --git a/tests/DBFixture.h b/tests/DBFixture.h
index be7e2e0..9efb330 100644 (file)
--- a/tests/DBFixture.h
+++ b/tests/DBFixture.h
@@ -1,5 +1,5 @@
 /*
- *  Copyright (c) 2016 Samsung Electronics Co., Ltd All Rights Reserved
+ *  Copyright (c) 2016-2019 Samsung Electronics Co., Ltd. All rights reserved
  *
  *  Contact: Kyungwook Tak <k.tak@samsung.com>
  *
@@ -30,7 +30,7 @@
 class DBFixture {
 public:
        DBFixture();
-       DBFixture(const char *db_fname);
+       explicit DBFixture(const char *db_fname);
 
        constexpr static const char *m_default_name = "name";
        constexpr static const char *m_default_owner = "label";

diff --git a/tests/test_comm-manager.cpp b/tests/test_comm-manager.cpp
index b06a3ac..6817632 100644 (file)
--- a/tests/test_comm-manager.cpp
+++ b/tests/test_comm-manager.cpp
@@ -1,5 +1,5 @@
 /*
- *  Copyright (c) 2000 - 2015 Samsung Electronics Co., Ltd All Rights Reserved
+ *  Copyright (c) 2000-2019 Samsung Electronics Co., Ltd. All rights reserved
  *
  *  Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
@@ -28,17 +28,17 @@
 
 namespace {
 struct MessageA {
-       MessageA(int ai) : i(ai) {}
+       explicit MessageA(int ai) : i(ai) {}
        int i;
 };
 
 struct MessageB {
-       MessageB(char ac) : c(ac) {}
+       explicit MessageB(char ac) : c(ac) {}
        char c;
 };
 
 struct MessageC {
-       MessageC(const std::string &astr) : str(astr) {}
+       explicit MessageC(const std::string &astr) : str(astr) {}
        std::string str;
 };
 

diff --git a/tests/test_safe-buffer.cpp b/tests/test_safe-buffer.cpp
index fa42c36..47ecbef 100644 (file)
--- a/tests/test_safe-buffer.cpp
+++ b/tests/test_safe-buffer.cpp
@@ -1,5 +1,5 @@
 /*
- *  Copyright (c) 2016 Samsung Electronics Co., Ltd All Rights Reserved
+ *  Copyright (c) 2016-2019 Samsung Electronics Co., Ltd. All rights reserved
  *
  *  Contact: Kyungwook Tak <k.tak@samsung.com>
  *
@@ -34,7 +34,7 @@ namespace {
 const size_t LEN = 100;
 
 struct Item {
-       Item(size_t a) : mA(a) {}
+       explicit Item(size_t a) : mA(a) {}
        ~Item() {}
 
        bool operator==(const size_t &other) const

diff --git a/tests/test_xml-parser.cpp b/tests/test_xml-parser.cpp
index 4b71217..5029306 100644 (file)
--- a/tests/test_xml-parser.cpp
+++ b/tests/test_xml-parser.cpp
@@ -1,5 +1,5 @@
 /*
- *  Copyright (c) 2000 - 2015 Samsung Electronics Co., Ltd All Rights Reserved
+ *  Copyright (c) 2000-2019 Samsung Electronics Co., Ltd. All rights reserved
  *
  *  Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
@@ -211,7 +211,7 @@ public:
                }
        };
 
-       StructureTest(const char *filename) : m_parser(filename), m_sum(0),
+       explicit StructureTest(const char *filename) : m_parser(filename), m_sum(0),
                m_expectedSum(0)
        {
                m_parser.RegisterErrorCb(StructureTest::Error);

diff --git a/tools/ckm_db_tool/db-crypto-ext.h b/tools/ckm_db_tool/db-crypto-ext.h
index bb1eb35..bbf7476 100644 (file)
--- a/tools/ckm_db_tool/db-crypto-ext.h
+++ b/tools/ckm_db_tool/db-crypto-ext.h
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2015-2017 Samsung Electronics Co., Ltd All Rights Reserved
+ * Copyright (c) 2015-2019 Samsung Electronics Co., Ltd. All rights reserved
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -30,7 +30,7 @@
 namespace CKM {
 namespace DB {
 struct CryptoExt : public Crypto {
-       CryptoExt(Crypto orig) : Crypto(std::move(orig)) {}
+       explicit CryptoExt(Crypto orig) : Crypto(std::move(orig)) {}
 
        SqlConnection::Output Execute(const std::string &cmd);
        RowVector getRows();