Avoid -Wnon-virtual-dtor warnings

This was SVN commit r7416.

source/collada/CommonConvert.h

@@ -43,6 +43,7 @@ private:
 
 struct OutputCB
 {
+	virtual ~OutputCB() { }
 	virtual void operator() (const char* data, unsigned int length)=0;
 };
 

source/lib/frequency_filter.h

@@ -26,6 +26,8 @@
 // calculate frequency of events (tuned for 100 Hz)
 struct IFrequencyFilter
 {
+	virtual ~IFrequencyFilter() {}
+
 	virtual void Update(double value) = 0;
 
 	// smoothed but rapidly tracked frequency

source/network/Network.h

@@ -95,6 +95,8 @@ class CMessageSocket;
 class IMessagePipeEnd
 {
 public:
+	virtual ~IMessagePipeEnd() {}
+
 	/**
 	 * Push a message on the output queue. It will be freed when popped of the
 	 * queue, not by the caller. The pointer must point to memory that can be

source/network/Serialization.h

@@ -75,6 +75,8 @@
 class ISerializable
 {
 public:
+	virtual ~ISerializable() {}
+
 	/**
 	 * Return the length of the serialized form of this object
 	 */

source/network/fsm.h

@@ -147,7 +147,7 @@ class CFsm
 public:
 
 	CFsm( void );
-	~CFsm( void );
+	virtual ~CFsm( void );
 
 	/**
 	 * Constructs the state machine. This method must be overriden so that

source/renderer/Scene.h

@@ -40,8 +40,11 @@ class SceneCollector;
  *
  * @see CRenderer::RenderScene
  */
-class Scene {
+class Scene
+{
 public:
+	virtual ~Scene() {}
+	
 	/**
 	 * Send all objects that can be seen when rendering the given frustum
 	 * to the scene collector.
@@ -58,8 +61,11 @@ public:
  *
  * @see Scene::EnumerateObjects
  */
-class SceneCollector {
+class SceneCollector
+{
 public:
+	virtual ~SceneCollector() {}
+
 	/**
 	 * Submit a terrain patch that is part of the scene.
 	 */

source/scripting/ScriptableObject.h

@@ -60,6 +60,7 @@ public:
 	virtual void AddProperty( const CStrW& PropertyName, const CStrW& Value ) = 0;
 	
 	inline IJSObject() {}
+	virtual ~IJSObject() {}
 };
 
 template<typename T, bool ReadOnly = false> class CJSObject;

source/simulation/AStarEngine.h

@@ -163,6 +163,7 @@ class AStarGoalBase
 {
 public:
 	AStarGoalBase() {}
+	virtual ~AStarGoalBase() {}
 	virtual void SetDestination( const CVector2D& ) = 0;
 	virtual void SetRadius( float r ) = 0;
 	virtual float DistanceToGoal( const CVector2D& ) = 0;

source/simulation/BoundingObjects.h

@@ -33,6 +33,7 @@ class CBoundingObject
 {
 public:
 	CBoundingObject() {}
+	virtual ~CBoundingObject() {}
 	enum EBoundingType
 	{
 		BOUND_NONE,

source/tools/atlas/AtlasUI/General/IAtlasSerialiser.h

@@ -26,6 +26,8 @@
 class IAtlasSerialiser
 {
 public:
+	virtual ~IAtlasSerialiser() { }
+
 	// Freeze/Thaw are mainly used by the 'undo' system, to take a snapshot
 	// of the a GUI component's state, and to revert to that snapshot.
 	// obj.Thaw(obj.Freeze()) should leave the component's contents unchanged.

source/tools/atlas/AtlasUI/ScenarioEditor/Tools/Common/Tools.h

@@ -111,6 +111,7 @@ protected:
 
 	struct State
 	{
+		virtual ~State() {}
 		virtual void OnEnter(T* WXUNUSED(obj)) {}
 		virtual void OnLeave(T* WXUNUSED(obj)) {}
 		virtual void OnTick (T* WXUNUSED(obj), float WXUNUSED(dt)) {}

source/tools/atlas/DatafileIO/Stream/Stream.h

@@ -38,7 +38,7 @@ namespace DatafileIO
 	public:
 		enum whence { FROM_START, FROM_END, FROM_CURRENT };
 
-		virtual ~Stream() {};
+		virtual ~Stream() {}
 		virtual off_t Tell() const = 0;
 		virtual bool IsOk() const = 0;
 	};
@@ -46,6 +46,7 @@ namespace DatafileIO
 	class SeekableStream
 	{
 	public:
+		virtual ~SeekableStream() {}
 		virtual void Seek(off_t pos, Stream::whence mode) = 0;
 	};
 

source/tools/atlas/GameInterface/MessagePasser.h

@@ -29,12 +29,14 @@ struct QueryMessage;
 class MessagePasser
 {
 public:
-	virtual void Add(IMessage*)=0;
+	virtual ~MessagePasser() {}
+
+	virtual void Add(IMessage*) = 0;
 		// takes ownership of IMessage object
 
-	virtual IMessage* Retrieve()=0;
+	virtual IMessage* Retrieve() = 0;
 
-	virtual void Query(QueryMessage*, void(*timeoutCallback)())=0;
+	virtual void Query(QueryMessage*, void(*timeoutCallback)()) = 0;
 		// blocks; caller retains ownership of QueryMessage object
 };