Remove -Wno-delete-non-virtual-dtor
authorTimePath <andrew.hardaker1995@gmail.com>
Sun, 6 Aug 2017 12:03:00 +0000 (22:03 +1000)
committerTimePath <andrew.hardaker1995@gmail.com>
Sun, 6 Aug 2017 12:03:00 +0000 (22:03 +1000)
23 files changed:
CMakeLists.txt
include/iarchive.h
include/ientity.h
include/iimage.h
include/imap.h
include/irender.h
include/iscenegraph.h
include/iscriplib.h
include/iselection.h
include/itextures.h
include/modelskin.h
include/moduleobserver.h
include/renderable.h
include/windowobserver.h
plugins/image/ktx.cpp
radiant/build.cpp
radiant/dialog.h
radiant/entityinspector.cpp
radiant/feedback.h
radiant/selection.cpp
radiant/selection.h
radiant/textures.cpp
radiant/xmlstuff.h

index 49029f3..65b7e26 100644 (file)
@@ -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})
index dddf557..b6cfda1 100644 (file)
@@ -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
 {
index c315b3a..db47ff4 100644 (file)
@@ -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;
index c159e65..7f355f2 100644 (file)
@@ -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;
index 7af4c47..396627d 100644 (file)
@@ -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;
index 5199454..8b0c3ec 100644 (file)
@@ -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;
index 6b28b0f..569cf86 100644 (file)
@@ -70,6 +70,7 @@ typedef Stack<NodeReference> Path;
 class Graph
 {
 public:
+virtual ~Graph() = default;
 INTEGER_CONSTANT( Version, 1 );
 STRING_CONSTANT( Name, "scenegraph" );
 
index 8f64503..e070406 100644 (file)
@@ -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;
index ba00002..a5951f9 100644 (file)
@@ -63,6 +63,7 @@ typedef SignalHandler1<const Selectable&> SelectionChangeHandler;
 class SelectionSystem
 {
 public:
+virtual ~SelectionSystem() = default;
 INTEGER_CONSTANT( Version, 1 );
 STRING_CONSTANT( Name, "selection" );
 
index cc91399..6fa4b02 100644 (file)
@@ -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;
index bb5922e..a2c219e 100644 (file)
@@ -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'.
index 496d6f2..0559b85 100644 (file)
@@ -25,6 +25,7 @@
 class ModuleObserver
 {
 public:
+virtual ~ModuleObserver() = default;
 virtual void unrealise() = 0;
 virtual void realise() = 0;
 };
index 0c0e770..d50ca7a 100644 (file)
@@ -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;
index b72d082..2494863 100644 (file)
@@ -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;
index 582d0b8..480ca9e 100644 (file)
@@ -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;
 };
index 6820981..b5a57f5 100644 (file)
@@ -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;
 };
index f4ce166..cf1c485 100644 (file)
@@ -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;
index 1266e9c..ff56706 100644 (file)
@@ -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;
index a927175..b683a31 100644 (file)
@@ -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
index 5013af7..e263191 100644 (file)
@@ -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;
 };
 
index 16b9aa2..1082c00 100644 (file)
@@ -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;
 };
index b223e2e..93ce42a 100644 (file)
@@ -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;
index b51f3cd..7fe6120 100644 (file)
@@ -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;