Remove -Wno-delete-non-virtual-dtor

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 49029f3652beac83bf5bd554c3a573dc88035388..65b7e26acf38703d517016b82fa380bc0fd7daf1 100644 (file)
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -99,8 +99,6 @@ addflags_c("-Wno-int-conversion")
 addflags_c("-Wno-pointer-to-int-cast")
 addflags_c("-Wno-incompatible-pointer-types")
 
-addflags_cxx("-Wno-delete-non-virtual-dtor")
-
 set(CMAKE_POSITION_INDEPENDENT_CODE 1)
 set(GTK_TARGET 2 CACHE STRING "GTK target")
 add_definitions(-DGTK_TARGET=${GTK_TARGET})

diff --git a/include/iarchive.h b/include/iarchive.h
index dddf5579853e5dac3ec916e15638efa7276b6710..b6cfda19d465f34b42b6bceef2c949226058ff1a 100644 (file)
--- a/include/iarchive.h
+++ b/include/iarchive.h
@@ -31,6 +31,7 @@ class InputStream;
 class ArchiveFile
 {
 public:
+virtual ~ArchiveFile() = default;
 /// \brief Destroys the file object.
 virtual void release() = 0;
 /// \brief Returns the size of the file data in bytes.
@@ -50,6 +51,7 @@ class TextInputStream;
 class ArchiveTextFile
 {
 public:
+virtual ~ArchiveTextFile() = default;
 /// \brief Destroys the file object.
 virtual void release() = 0;
 /// \brief Returns the stream associated with this file.
@@ -75,6 +77,7 @@ class CustomArchiveVisitor;
 class Archive
 {
 public:
+virtual ~Archive() = default;
 
 class Visitor
 {

diff --git a/include/ientity.h b/include/ientity.h
index c315b3abb2708a527c355ee7759efe44ef5d5363..db47ff498296b8c16278c41d57c630794a86df5e 100644 (file)
--- a/include/ientity.h
+++ b/include/ientity.h
@@ -34,6 +34,7 @@ typedef Callback1<const char*> KeyObserver;
 class EntityKeyValue
 {
 public:
+virtual ~EntityKeyValue() = default;
 virtual const char* c_str() const = 0;
 virtual void assign( const char* other ) = 0;
 virtual void attach( const KeyObserver& observer ) = 0;

diff --git a/include/iimage.h b/include/iimage.h
index c159e657152af1ba8ede594a4c3d53ffb99e234e..7f355f2ce17b8337f2219bf02ff5f3e51612dac8 100644 (file)
--- a/include/iimage.h
+++ b/include/iimage.h
@@ -29,6 +29,7 @@ typedef unsigned char byte;
 class Image
 {
 public:
+virtual ~Image() = default;
 virtual void release() = 0;
 virtual byte* getRGBAPixels() const = 0;
 virtual unsigned int getWidth() const = 0;

diff --git a/include/imap.h b/include/imap.h
index 7af4c4721335f9bb016b1f2ba2b8b58c9e7ab037..396627d0226b126c5d1c9061f9fcbb9958fff976 100644 (file)
--- a/include/imap.h
+++ b/include/imap.h
@@ -59,6 +59,7 @@ typedef void ( *GraphTraversalFunc )( scene::Node& root, const scene::Traversabl
 class MapFormat
 {
 public:
+virtual ~MapFormat() = default;
 INTEGER_CONSTANT( Version, 2 );
 STRING_CONSTANT( Name, "map" );
 mutable bool wrongFormat;

diff --git a/include/irender.h b/include/irender.h
index 5199454287cb9081797c61cfddc7a96461df0baf..8b0c3ec6b999507b594f646d89282c91b237f8ec 100644 (file)
--- a/include/irender.h
+++ b/include/irender.h
@@ -107,6 +107,7 @@ const int c_attr_Binormal = 4;
 class OpenGLRenderable
 {
 public:
+virtual ~OpenGLRenderable() = default;
 virtual void render( RenderStateFlags state ) const = 0;
 };
 
@@ -119,6 +120,7 @@ class ModuleObserver;
 class Shader
 {
 public:
+virtual ~Shader() = default;
 virtual void addRenderable( const OpenGLRenderable& renderable, const Matrix4& modelview, const LightList* lights = 0 ) = 0;
 virtual void incrementUsed() = 0;
 virtual void decrementUsed() = 0;

diff --git a/include/iscenegraph.h b/include/iscenegraph.h
index 6b28b0fc9a7122c3f5cf6184ebbd3263f120e987..569cf864c08a4e951ac8ef1b97a39e920637d731 100644 (file)
--- a/include/iscenegraph.h
+++ b/include/iscenegraph.h
@@ -70,6 +70,7 @@ typedef Stack<NodeReference> Path;
 class Graph
 {
 public:
+virtual ~Graph() = default;
 INTEGER_CONSTANT( Version, 1 );
 STRING_CONSTANT( Name, "scenegraph" );
 

diff --git a/include/iscriplib.h b/include/iscriplib.h
index 8f64503355b4ded1267a926dfd20a2dc4f3f42ab..e070406fc3fd6437697b4ca0f1c290bfb74eaa3f 100644 (file)
--- a/include/iscriplib.h
+++ b/include/iscriplib.h
@@ -33,6 +33,7 @@
 class Tokeniser
 {
 public:
+virtual ~Tokeniser() = default;
 virtual void release() = 0;
 virtual void nextLine() = 0;
 virtual const char* getToken() = 0;
@@ -46,6 +47,7 @@ class TextInputStream;
 class TokenWriter
 {
 public:
+virtual ~TokenWriter() = default;
 virtual void release() = 0;
 virtual void nextLine() = 0;
 virtual void writeToken( const char* token ) = 0;

diff --git a/include/iselection.h b/include/iselection.h
index ba000021a2c24dbe2fe07ce392897295d0b22105..a5951f9a78a031a6d1dc53fcf76296378641c9da 100644 (file)
--- a/include/iselection.h
+++ b/include/iselection.h
@@ -63,6 +63,7 @@ typedef SignalHandler1<const Selectable&> SelectionChangeHandler;
 class SelectionSystem
 {
 public:
+virtual ~SelectionSystem() = default;
 INTEGER_CONSTANT( Version, 1 );
 STRING_CONSTANT( Name, "selection" );
 

diff --git a/include/itextures.h b/include/itextures.h
index cc91399d2c452115b233e660ad5ab9d5fe139e43..6fa4b02ca51e6c9d7200483cd87248a1c411b33c 100644 (file)
--- a/include/itextures.h
+++ b/include/itextures.h
@@ -59,6 +59,7 @@ virtual void realise() = 0;
 class TexturesCache
 {
 public:
+virtual ~TexturesCache() = default;
 INTEGER_CONSTANT( Version, 1 );
 STRING_CONSTANT( Name, "textures" );
 virtual LoadImageCallback defaultLoader() const = 0;

diff --git a/include/modelskin.h b/include/modelskin.h
index bb5922ee7704b89c48ee666a8c5d353dd9e1b1b2..a2c219e6be1ca48c85d53e58b28ea3cacf1a5ff4 100644 (file)
--- a/include/modelskin.h
+++ b/include/modelskin.h
@@ -40,6 +40,7 @@ class ModuleObserver;
 class ModelSkin
 {
 public:
+virtual ~ModelSkin() = default;
 STRING_CONSTANT( Name, "ModelSkin" );
 /// \brief Attach an \p observer whose realise() and unrealise() methods will be called when the skin is loaded or unloaded.
 virtual void attach( ModuleObserver& observer ) = 0;
@@ -64,6 +65,7 @@ virtual void skinChanged() = 0;
 class ModelSkinCache
 {
 public:
+virtual ~ModelSkinCache() = default;
 INTEGER_CONSTANT( Version, 1 );
 STRING_CONSTANT( Name, "modelskin" );
 /// \brief Increments the reference count of and returns a reference to the skin uniquely identified by 'name'.

diff --git a/include/moduleobserver.h b/include/moduleobserver.h
index 496d6f2e930d402eb9fdb2f7470e7d31e479672e..0559b850bdd308ffd602b8b0bf35a72105ac6637 100644 (file)
--- a/include/moduleobserver.h
+++ b/include/moduleobserver.h
@@ -25,6 +25,7 @@
 class ModuleObserver
 {
 public:
+virtual ~ModuleObserver() = default;
 virtual void unrealise() = 0;
 virtual void realise() = 0;
 };

diff --git a/include/renderable.h b/include/renderable.h
index 0c0e770bd35347719aac73338019edbbb1e88ca2..d50ca7a8f57c87e4616f4cd6e5c14b5335d53c23 100644 (file)
--- a/include/renderable.h
+++ b/include/renderable.h
@@ -60,6 +60,7 @@ class VolumeTest;
 class Renderable
 {
 public:
+virtual ~Renderable() = default;
 STRING_CONSTANT( Name, "Renderable" );
 
 virtual void renderSolid( Renderer& renderer, const VolumeTest& volume ) const = 0;

diff --git a/include/windowobserver.h b/include/windowobserver.h
index b72d0820682b4d458e1290e20f985b1e4914632d..24948634ca1d05e2c7d3f72b1bd689cf25ac6e0d 100644 (file)
--- a/include/windowobserver.h
+++ b/include/windowobserver.h
@@ -79,6 +79,7 @@ typedef Vector2 WindowVector;
 class WindowObserver
 {
 public:
+virtual ~WindowObserver() = default;
 virtual void release() = 0;
 virtual void onSizeChanged( int width, int height ) = 0;
 virtual void onMouseDown( const WindowVector& position, ButtonIdentifier button, ModifierFlags modifiers ) = 0;

diff --git a/plugins/image/ktx.cpp b/plugins/image/ktx.cpp
index 582d0b8b3652eeee74b321c3522e5bd983e09126..480ca9e8c65e15a4ce870670dae576b78eb686dc 100644 (file)
--- a/plugins/image/ktx.cpp
+++ b/plugins/image/ktx.cpp
@@ -47,6 +47,7 @@
 class KTX_Decoder
 {
 public:
+       virtual ~KTX_Decoder() = default;
        virtual void Decode( PointerInputStream& istream, byte* out ) = 0;
        virtual unsigned int GetPixelSize() = 0;
 };

diff --git a/radiant/build.cpp b/radiant/build.cpp
index 6820981bf091230d73686fcc3b271f91b3b184fb..b5a57f5eb1d8ca2bfa423960effd41fde66ae923 100644 (file)
--- a/radiant/build.cpp
+++ b/radiant/build.cpp
@@ -56,6 +56,7 @@ const char* build_get_variable( const char* name ){
 class Evaluatable
 {
 public:
+virtual ~Evaluatable() = default;
 virtual void evaluate( StringBuffer& output ) = 0;
 virtual void exportXML( XMLImporter& importer ) = 0;
 };
@@ -172,6 +173,7 @@ void exportXML( XMLImporter& importer ){
 class XMLElementParser : public TextOutputStream
 {
 public:
+virtual ~XMLElementParser() = default;
 virtual XMLElementParser& pushElement( const XMLElement& element ) = 0;
 virtual void popElement( const char* name ) = 0;
 };

diff --git a/radiant/dialog.h b/radiant/dialog.h
index f4ce16673e7f7f8ffbe4fac55acfa87567587e54..cf1c4858913df592aaf7285a76438d22da45fac1 100644 (file)
--- a/radiant/dialog.h
+++ b/radiant/dialog.h
@@ -88,6 +88,7 @@ typedef ReferenceCaller1<CopiedString, const StringImportCallback&, StringExport
 
 struct DLG_DATA
 {
+       virtual ~DLG_DATA() = default;
        virtual void release() = 0;
        virtual void importData() const = 0;
        virtual void exportData() const = 0;

diff --git a/radiant/entityinspector.cpp b/radiant/entityinspector.cpp
index 1266e9c6e690c8620e4cd2e023641c60f74b3f2c..ff56706001126681a42a49c2b7e6b4dc4d4110e9 100644 (file)
--- a/radiant/entityinspector.cpp
+++ b/radiant/entityinspector.cpp
@@ -104,6 +104,7 @@ void Scene_EntitySetKeyValue_Selected_Undoable( const char* key, const char* val
 class EntityAttribute
 {
 public:
+virtual ~EntityAttribute() = default;
 virtual ui::Widget getWidget() const = 0;
 virtual void update() = 0;
 virtual void release() = 0;

diff --git a/radiant/feedback.h b/radiant/feedback.h
index a927175566394d20fc82d6ae69161885fea4ee49..b683a316d76ea07abe1cc48b43c238b36614b68e 100644 (file)
--- a/radiant/feedback.h
+++ b/radiant/feedback.h
@@ -39,6 +39,7 @@
 class IGL2DWindow
 {
 public:
+virtual ~IGL2DWindow() = default;
 // Increment the number of references to this object
 virtual void IncRef() = 0;
 // Decrement the reference count

diff --git a/radiant/selection.cpp b/radiant/selection.cpp
index 5013af7ba466d3b66840e10fbcb02cd6b54df667..e26319170992d0364cb0d337fc8262785b0daafd 100644 (file)
--- a/radiant/selection.cpp
+++ b/radiant/selection.cpp
@@ -206,6 +206,7 @@ void transform_local2object( Matrix4& object, const Matrix4& local, const Matrix
 class Rotatable
 {
 public:
+virtual ~Rotatable() = default;
 virtual void rotate( const Quaternion& rotation ) = 0;
 };
 
@@ -270,6 +271,7 @@ void translation_local2object( Vector3& object, const Vector3& local, const Matr
 class Translatable
 {
 public:
+virtual ~Translatable() = default;
 virtual void translate( const Vector3& translation ) = 0;
 };
 
@@ -329,6 +331,7 @@ void Transform( const Matrix4& manip2object, const Matrix4& device2manip, const
 class Scalable
 {
 public:
+virtual ~Scalable() = default;
 virtual void scale( const Vector3& scaling ) = 0;
 };
 

diff --git a/radiant/selection.h b/radiant/selection.h
index 16b9aa2f82cf46a470c5d4594919f512cf1b1cdd..1082c002d5f2ac6962db55ff06c8dee6a8f867a6 100644 (file)
--- a/radiant/selection.h
+++ b/radiant/selection.h
@@ -38,6 +38,7 @@ class View;
 class SelectionSystemWindowObserver : public WindowObserver
 {
 public:
+virtual ~SelectionSystemWindowObserver() = default;
 virtual void setView( const View& view ) = 0;
 virtual void setRectangleDrawCallback( const RectangleCallback& callback ) = 0;
 };

diff --git a/radiant/textures.cpp b/radiant/textures.cpp
index b223e2ee1318a87c40d0b7ccbab788f13a72f1c1..93ce42a8cdbec549f21f9b64926896c5f357d650 100644 (file)
--- a/radiant/textures.cpp
+++ b/radiant/textures.cpp
@@ -409,6 +409,7 @@ TexturesCacheObserver* m_observer;
 std::size_t m_unrealised;
 
 public:
+virtual ~TexturesMap() = default;
 TexturesMap() : m_qtextures( TextureConstructor( this ) ), m_observer( 0 ), m_unrealised( 1 ){
 }
 typedef qtextures_t::iterator iterator;

diff --git a/radiant/xmlstuff.h b/radiant/xmlstuff.h
index b51f3cd305e1a38a013ec7d2809375d06b1405cb..7fe6120f6031b46fa531744bc2db7bfc29236412 100644 (file)
--- a/radiant/xmlstuff.h
+++ b/radiant/xmlstuff.h
@@ -68,6 +68,7 @@ class IGL2DWindow;
 class ISAXHandler
 {
 public:
+virtual ~ISAXHandler() = default;
 virtual void Release(){}
 virtual void saxStartElement( message_info_t* ctx, const xmlChar* name, const xmlChar** attrs ) = 0;
 virtual void saxEndElement( message_info_t* ctx, const xmlChar* name ) = 0;