Change default virtual destructor from '= default' to '{}' 80/25380/3
authorAdam Malinowski <a.malinowsk2@partner.samsung.com>
Mon, 4 Aug 2014 12:12:13 +0000 (14:12 +0200)
committerAdam Malinowski <a.malinowsk2@partner.samsung.com>
Tue, 5 Aug 2014 07:55:37 +0000 (09:55 +0200)
'= default' seems to be problematic. Some compilers give error:
"declared virtual cannot be defaulted in the class body".

Change-Id: Iaca3a70f64e45309430bc010883a87fcdc536d1b

42 files changed:
src/admin/api/ApiInterface.h
src/admin/logic/Logic.h
src/client/api/ApiInterface.h
src/client/cache/CacheInterface.h
src/client/logic/Logic.h
src/common/exceptions/BucketDeserializationException.h
src/common/exceptions/BucketNotExistsException.h
src/common/exceptions/BucketRecordCorruptedException.h
src/common/exceptions/BucketSerializationException.h
src/common/exceptions/CannotCreateFileException.h
src/common/exceptions/DefaultBucketDeletionException.h
src/common/exceptions/DescriptorNotExistsException.h
src/common/exceptions/Exception.h
src/common/exceptions/FileNotFoundException.h
src/common/exceptions/InitException.h
src/common/exceptions/InvalidProtocolException.h
src/common/exceptions/NotImplementedException.h
src/common/exceptions/NullPointerException.h
src/common/exceptions/OutOfDataException.h
src/common/exceptions/PluginNotFoundException.h
src/common/exceptions/ServerConnectionErrorException.h
src/common/exceptions/UnexpectedErrorException.h
src/common/protocol/Protocol.h
src/common/protocol/ProtocolFrame.h
src/common/protocol/ProtocolFrameHeader.h
src/common/protocol/ProtocolSerialization.h
src/common/request/CheckRequest.h
src/common/request/InsertOrUpdateBucketRequest.h
src/common/request/RemoveBucketRequest.h
src/common/request/Request.h
src/common/request/RequestTaker.h
src/common/request/SetPoliciesRequest.h
src/common/request/SignalRequest.h
src/common/response/CheckResponse.h
src/common/response/CodeResponse.h
src/common/response/Response.h
src/common/response/ResponseTaker.h
src/common/sockets/SocketClient.h
src/service/storage/InMemoryStorageBackend.h
src/service/storage/StorageSerializer.h
test/storage/serializer/bucket_load.cpp
test/storage/serializer/serialize.cpp

index 9c513f6..eb2b17b 100644 (file)
@@ -39,7 +39,7 @@ namespace Cynara {
 class ApiInterface {
 public:
     ApiInterface() = default;
-    virtual ~ApiInterface() = default;
+    virtual ~ApiInterface() {};
 
     virtual int setPolicies(const std::map<PolicyBucketId, std::vector<Policy>> &insertOrUpdate,
                     const std::map<PolicyBucketId, std::vector<PolicyKey>> &remove) noexcept = 0;
index d61c7e8..2d349f3 100644 (file)
@@ -40,7 +40,7 @@ private:
 
 public:
     Logic();
-    virtual ~Logic() = default;
+    virtual ~Logic() {};
 
     virtual int setPolicies(const std::map<PolicyBucketId, std::vector<Policy>> &insertOrUpdate,
                     const std::map<PolicyBucketId, std::vector<PolicyKey>> &remove) noexcept;
index f268508..73bcec5 100644 (file)
@@ -31,7 +31,7 @@ namespace Cynara {
 class ApiInterface {
 public:
     ApiInterface() = default;
-    virtual ~ApiInterface() = default;
+    virtual ~ApiInterface() {};
 
     virtual int check(const std::string &client, const std::string &session,
                       const std::string &user, const std::string &privilege) = 0;
index f325a42..0279fdb 100644 (file)
@@ -47,7 +47,7 @@ typedef std::shared_ptr<ResultGetterInterface> ResultGetterInterfacePtr;
 class ResultGetterInterface {
 public:
     virtual int requestResult(const PolicyKey &key, PolicyResult &result) noexcept = 0;
-    virtual ~ResultGetterInterface() = default;
+    virtual ~ResultGetterInterface() {};
 };
 
 class InterpreterInterface {
@@ -56,7 +56,7 @@ public:
     virtual bool isUsable(const PolicyResult &result) noexcept = 0;
     virtual int toResult(const PolicyResult &result) noexcept = 0;
 
-    virtual ~InterpreterInterface() = default;
+    virtual ~InterpreterInterface() {};
 };
 
 class PluginCache {
@@ -72,7 +72,7 @@ public:
         m_plugins.clear();
     }
 
-    virtual ~PluginCache() = default;
+    virtual ~PluginCache() {};
 
 protected:
     std::map<PolicyType, InterpreterInterfacePtr> m_plugins;
index e9f2c23..ec298da 100644 (file)
@@ -41,7 +41,7 @@ private:
 
 public:
     Logic();
-    virtual ~Logic() = default;
+    virtual ~Logic() {};
 
     virtual int check(const std::string &client, const std::string &session,
                       const std::string &user, const std::string &privilege) noexcept;
index 2a00555..01d9a80 100644 (file)
@@ -30,6 +30,7 @@ namespace Cynara {
 class BucketDeserializationException : public DatabaseException {
 public:
     BucketDeserializationException(const PolicyBucketId &bucketId) : m_bucketId(bucketId) {}
+    ~BucketDeserializationException() noexcept {};
 
     const std::string message(void) const {
         if (m_message.empty()) {
index 54c942e..18eb190 100644 (file)
@@ -34,7 +34,7 @@ class BucketNotExistsException : public Exception {
 public:
     BucketNotExistsException() = delete;
     BucketNotExistsException(const PolicyBucketId &bucketId) : m_bucketId(bucketId) {}
-    virtual ~BucketNotExistsException() = default;
+    virtual ~BucketNotExistsException() noexcept {};
 
     virtual const std::string message(void) const {
         return "BucketNotExistsException";
index 227c1d9..8152a77 100644 (file)
@@ -31,7 +31,7 @@ namespace Cynara {
 class BucketRecordCorruptedException : public Exception {
 public:
     BucketRecordCorruptedException(void) = delete;
-    virtual ~BucketRecordCorruptedException(void) = default;
+    virtual ~BucketRecordCorruptedException() noexcept {};
 
     BucketRecordCorruptedException(const std::string &line)
         : m_lineNumber(0), m_line(line)  {}
index 8882ff4..83f3397 100644 (file)
@@ -30,6 +30,7 @@ namespace Cynara {
 class BucketSerializationException : public DatabaseException {
 public:
     BucketSerializationException(const PolicyBucketId &bucketId) : m_bucketId(bucketId) {}
+    ~BucketSerializationException() noexcept {};
 
     const std::string message(void) const {
         if (m_message.empty()) {
index bb3f937..cfdfb49 100644 (file)
@@ -32,7 +32,7 @@ namespace Cynara {
 class CannotCreateFileException : public DatabaseException {
 public:
     CannotCreateFileException(const std::string &filename) : m_filename(filename) {};
-    virtual ~CannotCreateFileException() = default;
+    virtual ~CannotCreateFileException() noexcept {};
 
     const std::string message(void) const {
         if (m_message.empty()) {
index 6a92ec5..92e1a73 100644 (file)
@@ -32,7 +32,7 @@ namespace Cynara {
 class DefaultBucketDeletionException : public Exception {
 public:
     DefaultBucketDeletionException() = default;
-    virtual ~DefaultBucketDeletionException() = default;
+    virtual ~DefaultBucketDeletionException() noexcept {};
 
     virtual const std::string message(void) const {
         return "DefaultBucketDeletionException";
index 5dd9553..095ffba 100644 (file)
@@ -43,7 +43,7 @@ public:
         m_whatMsg = stream.str();
     }
 
-    virtual ~DescriptorNotExistsException() = default;
+    virtual ~DescriptorNotExistsException() noexcept {};
 
     virtual const std::string message(void) const {
         return m_whatMsg;
index b709060..74ded47 100644 (file)
@@ -33,7 +33,7 @@ public:
         m_backtrace = Backtrace::getBacktrace();
     }
 
-    virtual ~Exception() = default;
+    virtual ~Exception() noexcept {};
 
     virtual const char *what(void) const noexcept {
         if(m_whatMessage.empty()) {
index e353c19..bce1fcb 100644 (file)
@@ -32,7 +32,7 @@ namespace Cynara {
 class FileNotFoundException : public DatabaseException {
 public:
     FileNotFoundException(const std::string &filename) : m_filename(filename) {};
-    virtual ~FileNotFoundException() = default;
+    virtual ~FileNotFoundException() noexcept {};
 
     const std::string message(void) const {
         if (m_message.empty()) {
index d06e1d7..c51cb5b 100644 (file)
@@ -32,7 +32,7 @@ namespace Cynara {
 class InitException : public Exception {
 public:
     InitException() = default;
-    virtual ~InitException() = default;
+    virtual ~InitException() noexcept {};
 
     virtual const std::string message(void) const {
         return "InitException";
index a2d65d1..33053e9 100644 (file)
@@ -59,7 +59,7 @@ public:
 
     }
 
-    virtual ~InvalidProtocolException() = default;
+    virtual ~InvalidProtocolException() noexcept {};
 
     virtual const std::string message(void) const {
         return m_whatMessage;
index ceabb9a..3d1882a 100644 (file)
@@ -32,7 +32,7 @@ namespace Cynara {
 class NotImplementedException : public Exception {
 public:
     NotImplementedException() = default;
-    virtual ~NotImplementedException() = default;
+    virtual ~NotImplementedException() noexcept {};
 
     virtual const std::string message(void) const {
         return "NotImplementedException";
index ffd0bef..02db1c5 100644 (file)
@@ -42,7 +42,7 @@ public:
                   + std::string(">");
     }
 
-    virtual ~NullPointerException() = default;
+    virtual ~NullPointerException() noexcept {};
 
     virtual const std::string message(void) const {
         return m_whatMsg;
index 827eb7a..c0b63bd 100644 (file)
@@ -44,7 +44,7 @@ public:
         m_whatMsg = stream.str();
     }
 
-    virtual ~OutOfDataException() = default;
+    virtual ~OutOfDataException() noexcept {};
 
     virtual const std::string message(void) const {
         return m_whatMsg;
index 345fd1d..4e422d3 100644 (file)
@@ -45,7 +45,7 @@ public:
         m_whatMessage = stream.str();
     }
 
-    virtual ~PluginNotFoundException() = default;
+    virtual ~PluginNotFoundException() noexcept {};
 
     virtual const std::string message(void) const {
         return m_whatMessage;
index a5a7b08..88a5e47 100644 (file)
@@ -32,7 +32,7 @@ namespace Cynara {
 class ServerConnectionErrorException : public Exception {
 public:
     ServerConnectionErrorException() = default;
-    virtual ~ServerConnectionErrorException() = default;
+    virtual ~ServerConnectionErrorException() noexcept {};
     virtual const std::string message(void) const {
         return "ServerConnectionError";
     }
index a7d100b..f3fca09 100644 (file)
@@ -49,7 +49,7 @@ public:
         m_whatMessage = stream.str();
     }
 
-    virtual ~UnexpectedErrorException() = default;
+    virtual ~UnexpectedErrorException() noexcept {};
 
     virtual const std::string message(void) const {
         return m_whatMessage;
index daf001c..5069cff 100644 (file)
@@ -40,7 +40,7 @@ typedef std::shared_ptr<Protocol> ProtocolPtr;
 class Protocol : public RequestTaker, public ResponseTaker {
 public:
     Protocol() = default;
-    virtual ~Protocol() = default;
+    virtual ~Protocol() {};
 
     virtual ProtocolPtr clone(void) = 0;
 
index 709196e..f3c64b8 100644 (file)
@@ -40,7 +40,7 @@ class ProtocolFrame: public IStream {
 
 public:
     ProtocolFrame(ProtocolFrameHeaderPtr frameHeader, BinaryQueuePtr headerContent);
-    virtual ~ProtocolFrame() = default;
+    virtual ~ProtocolFrame() {};
 
     ProtocolFrameHeaderPtr frameHeader(void) {
         return m_frameHeader;
index 0db0d6b..6560255 100644 (file)
@@ -46,7 +46,7 @@ private:
 
 public:
     ProtocolFrameHeader(BinaryQueuePtr headerContent = nullptr);
-    virtual ~ProtocolFrameHeader() = default;
+    virtual ~ProtocolFrameHeader() {};
 
     virtual void read(size_t num, void *bytes);
     virtual void write(size_t num, const void *bytes);
index 532f78b..f3acff7 100644 (file)
@@ -38,7 +38,7 @@ class IStream {
 public:
     virtual void read(size_t num, void *bytes) = 0;
     virtual void write(size_t num, const void *bytes) = 0;
-    virtual ~IStream() = default;
+    virtual ~IStream() {};
 };
 
 // Serializable interface
@@ -47,7 +47,7 @@ public:
     /*    ISerializable(){};
      *    ISerializable(IStream&){}; */
     virtual void serialize(IStream &) const = 0;
-    virtual ~ISerializable() = default;
+    virtual ~ISerializable() {};
 };
 
 struct ProtocolSerialization {
index 03135c7..7262347 100644 (file)
@@ -40,7 +40,7 @@ public:
         Request(sequenceNumber), m_key(key) {
     }
 
-    virtual ~CheckRequest() = default;
+    virtual ~CheckRequest() {};
 
     const PolicyKey &key(void) const {
         return m_key;
index b166d71..2cf03ce 100644 (file)
@@ -43,7 +43,7 @@ public:
         Request(sequenceNumber), m_bucketId(bucketId), m_result(result) {
     }
 
-    virtual ~InsertOrUpdateBucketRequest() = default;
+    virtual ~InsertOrUpdateBucketRequest() {};
 
     const PolicyBucketId &bucketId(void) const {
         return m_bucketId;
index ff49a43..49ef2a0 100644 (file)
@@ -40,7 +40,7 @@ public:
         : Request(sequenceNumber), m_bucketId(bucketId) {
     }
 
-    virtual ~RemoveBucketRequest() = default;
+    virtual ~RemoveBucketRequest() {};
 
     const PolicyBucketId &bucketId(void) const {
         return m_bucketId;
index ac92398..9b9a2d2 100644 (file)
@@ -33,7 +33,7 @@ class Request {
 public:
     Request(ProtocolFrameSequenceNumber sequenceNumber) : m_sequenceNumber(sequenceNumber) {
     }
-    virtual ~Request() = default;
+    virtual ~Request() {};
 
     virtual void execute(RequestPtr self, RequestTakerPtr taker,
             RequestContextPtr context) const = 0;
index 0c3ff7d..9d0a10f 100644 (file)
@@ -30,7 +30,7 @@ namespace Cynara {
 class RequestTaker {
 public:
     RequestTaker() = default;
-    virtual ~RequestTaker() = default;
+    virtual ~RequestTaker() {};
 
     virtual void execute(RequestContextPtr context, CheckRequestPtr request);
     virtual void execute(RequestContextPtr context, InsertOrUpdateBucketRequestPtr request);
index d876ad1..afb8f14 100644 (file)
@@ -49,7 +49,7 @@ public:
         m_removePolicies(removePolicies) {
     }
 
-    virtual ~SetPoliciesRequest() = default;
+    virtual ~SetPoliciesRequest() {};
 
     const std::map<PolicyBucketId, std::vector<Policy>> &policiesToBeInsertedOrUpdated(void) const {
         return m_insertOrUpdatePolicies;
index f9d1422..068834d 100644 (file)
@@ -38,7 +38,7 @@ public:
     SignalRequest(struct signalfd_siginfo sigInfo) : Request(0), m_sigInfo(sigInfo) {
     }
 
-    virtual ~SignalRequest() = default;
+    virtual ~SignalRequest() {};
 
     virtual void execute(RequestPtr self, RequestTakerPtr taker, RequestContextPtr context) const;
 
index 1f49648..1ca69f3 100644 (file)
@@ -39,7 +39,7 @@ public:
         Response(sequenceNumber), m_resultRef(result) {
     }
 
-    virtual ~CheckResponse() = default;
+    virtual ~CheckResponse() {};
 
     virtual void execute(ResponsePtr self, ResponseTakerPtr taker,
                          RequestContextPtr context) const;
index ec04e03..96d04c6 100644 (file)
@@ -43,7 +43,7 @@ public:
         Response(sequenceNumber), m_code(code) {
     }
 
-    virtual ~CodeResponse() = default;
+    virtual ~CodeResponse() {};
 
     virtual void execute(ResponsePtr self, ResponseTakerPtr taker,
                          RequestContextPtr context) const;
index e85de48..0731975 100644 (file)
@@ -33,7 +33,7 @@ class Response {
 public:
     Response(ProtocolFrameSequenceNumber sequenceNumber) : m_sequenceNumber(sequenceNumber) {
     };
-    virtual ~Response() = default;
+    virtual ~Response() {};
 
     virtual void execute(ResponsePtr self, ResponseTakerPtr taker,
                          RequestContextPtr context) const = 0;
index 545f60f..8aede4d 100644 (file)
@@ -31,7 +31,7 @@ namespace Cynara {
 class ResponseTaker {
 public:
     ResponseTaker() = default;
-    virtual ~ResponseTaker() = default;
+    virtual ~ResponseTaker() {};
 
     virtual void execute(RequestContextPtr context, CheckResponsePtr response);
     virtual void execute(RequestContextPtr context, CodeResponsePtr response);
index fae5ba3..7553a5a 100644 (file)
@@ -46,7 +46,7 @@ private:
 
 public:
     SocketClient(const std::string &socketPath, ProtocolPtr protocol);
-    virtual ~SocketClient() = default;
+    virtual ~SocketClient() {};
 
     //returns pointer to response
     //        or nullptr when connection to cynara service is lost
index be187bb..d811edd 100644 (file)
@@ -44,7 +44,7 @@ class InMemoryStorageBackend : public StorageBackend {
 public:
     InMemoryStorageBackend(const std::string &path) : m_dbPath(path) {
     }
-    virtual ~InMemoryStorageBackend() = default;
+    virtual ~InMemoryStorageBackend() {};
 
     virtual void load(void);
     virtual void save(void);
index a995265..55ec2cd 100644 (file)
@@ -45,7 +45,7 @@ public:
             BucketStreamOpener;
 
     StorageSerializer(std::shared_ptr<std::ostream> os);
-    virtual ~StorageSerializer() = default;
+    virtual ~StorageSerializer() {};
 
     virtual void dump(const Buckets &buckets,
                       BucketStreamOpener streamOpener);
index 3c2096e..8e5b2df 100644 (file)
@@ -43,7 +43,7 @@ MATCHER_P(PolicyAtPtrEq, policy, "") {
 
 class BucketDeserializerFixture : public ::testing::Test {
 public:
-    virtual ~BucketDeserializerFixture() = default;
+    virtual ~BucketDeserializerFixture() {};
 
     PolicyPtr createPolicy(const PolicyKey &pk, const PolicyResult &pr) {
         return std::make_shared<Policy>(pk, pr);
index 07804f4..ad55495 100644 (file)
@@ -56,7 +56,7 @@ public:
 
 class StorageSerializerFixture : public ::testing::Test {
 public:
-    virtual ~StorageSerializerFixture() = default;
+    virtual ~StorageSerializerFixture() {};
 
     Cynara::Buckets buckets;
     FakeStreamForBucketId fakeStreamOpener;