From d7e4844d85db591a2ea235dd1b081c76e2f5cb9d Mon Sep 17 00:00:00 2001 From: Sebastien Gonzalve Date: Tue, 12 Mar 2013 14:30:27 +0100 Subject: Remove no-non-virtual-dtor flag BZ: 111473 The warning non-virtual-dtor should not be ignored for the whole project, but only in the few cases where it it necessary. Change-Id: I0ede82a30f2cd055b789f7fb56104fb227934e6f Signed-off-by: Sebastien Gonzalve Signed-off-by: Guilhem IMBERTON Reviewed-on: http://android.intel.com:8080/110191 Reviewed-by: buildbot Tested-by: buildbot --- remote-processor/Android.mk | 3 --- remote-processor/RemoteCommand.h | 3 +++ remote-processor/RemoteCommandHandler.h | 3 +++ remote-processor/RemoteCommandHandlerTemplate.h | 1 + remote-processor/RemoteProcessorServerInterface.h | 3 +++ 5 files changed, 10 insertions(+), 3 deletions(-) (limited to 'remote-processor') diff --git a/remote-processor/Android.mk b/remote-processor/Android.mk index 7da064a..2a79297 100644 --- a/remote-processor/Android.mk +++ b/remote-processor/Android.mk @@ -16,10 +16,7 @@ COMMON_SRC_FILES := \ COMMON_MODULE := libremote-processor COMMON_MODULE_TAGS := optional -COMMON_CFLAGS := -Wno-non-virtual-dtor - COMMON_LDLIBS := -lpthread - ############################# # Target build diff --git a/remote-processor/RemoteCommand.h b/remote-processor/RemoteCommand.h index df3d5a4..ffcbcfc 100644 --- a/remote-processor/RemoteCommand.h +++ b/remote-processor/RemoteCommand.h @@ -38,4 +38,7 @@ public: virtual uint32_t getArgumentCount() const = 0; virtual const std::string& getArgument(uint32_t uiArgument) const = 0; virtual const std::string packArguments(uint32_t uiStartArgument, uint32_t uiNbArguments) const = 0; + +protected: + virtual ~IRemoteCommand() {} }; diff --git a/remote-processor/RemoteCommandHandler.h b/remote-processor/RemoteCommandHandler.h index 97aa04f..fcd98a9 100644 --- a/remote-processor/RemoteCommandHandler.h +++ b/remote-processor/RemoteCommandHandler.h @@ -32,4 +32,7 @@ class IRemoteCommandHandler public: // Return true on success, fill result in any cases virtual bool remoteCommandProcess(const IRemoteCommand& remoteCommand, std::string& strResult) = 0; + +protected: + virtual ~IRemoteCommandHandler() {} }; diff --git a/remote-processor/RemoteCommandHandlerTemplate.h b/remote-processor/RemoteCommandHandlerTemplate.h index 5d9817e..6223b9c 100644 --- a/remote-processor/RemoteCommandHandlerTemplate.h +++ b/remote-processor/RemoteCommandHandlerTemplate.h @@ -217,3 +217,4 @@ private: std::vector _remoteCommandParserVector; uint32_t _uiMaxCommandUsageLength; }; + diff --git a/remote-processor/RemoteProcessorServerInterface.h b/remote-processor/RemoteProcessorServerInterface.h index 11ceb10..fdc97cc 100644 --- a/remote-processor/RemoteProcessorServerInterface.h +++ b/remote-processor/RemoteProcessorServerInterface.h @@ -34,4 +34,7 @@ public: virtual bool start() = 0; virtual void stop() = 0; virtual bool isStarted() const = 0; + + /* FIXME this was missing but is explicitly called */ + virtual ~IRemoteProcessorServerInterface() {} }; -- cgit v1.1