Changed: #825 Remove all warnings when compiling Ryzom

code/ryzom/common/src/game_share/singleton_registry.h

@@ -58,6 +58,7 @@ protected:
 	// protect from untrolled instantiation
 	// this method registers the singleton with the singleton registry
 	IServiceSingleton();
+	virtual ~IServiceSingleton() {}
 
 private:
 	// prohibit copy

code/ryzom/server/src/entities_game_service/pvp_manager/pvp_interface.h

@@ -38,6 +38,7 @@ class CGameItemPtr;
 class IPVPInterface : public NLMISC::CRefCount
 {
 public:
+	virtual ~IPVPInterface() {}
 	///\name PVP MODE FEATURES DEPENDANTS
 	//@{
 	/// return pvp relation between the two players
@@ -61,8 +62,6 @@ public:
 	/// killed character in PvP faction
 	virtual void characterKilledInPvPFaction( CCharacter * character, PVP_CLAN::TPVPClan looserFaction, sint32 factionPoint ) const = 0;
 	//@}
-
-private:
 };
 
 #endif // RY_PVP_INTERFACE_H

code/ryzom/server/src/general_utilities_service/gus_client_manager.h

@@ -93,6 +93,7 @@ namespace GUS
 		class IConnectionHandler: public NLMISC::CRefCount
 		{
 		public:
+			virtual ~IConnectionHandler() {}
 			virtual void connect(TClientId)=0;
 			virtual void disconnect(TClientId)=0;
 		};

code/ryzom/server/src/general_utilities_service/rs_remote_saves.h

@@ -68,6 +68,8 @@ namespace SAVES
 	class CRemoteSavesManager: public NLMISC::CRefCount
 	{
 	public:
+		virtual ~CRemoteSavesManager() {}
+	
 		static CRemoteSavesManager* getInstance();
 
 		// interface used by CRemoteSavesInterface objects in their ctor to declare themselves

code/ryzom/server/src/general_utilities_service/saves_unit.h

@@ -141,6 +141,9 @@ namespace SAVES
 		// ctor
 		ISavesUnitElement();
 
+		// dtor
+		virtual ~ISavesUnitElement() {}
+
 		// perform the 'rescan' operation for this element - verify whether anything has changed
 		// the parent parameter is used to supply addNew(), addChange() and addDeleted() methods
 		// to signal any detected changes

code/ryzom/server/src/patchman_service/patchman_tester.h

@@ -38,6 +38,8 @@ namespace PATCHMAN
 	class CPatchmanTester
 	{
 	public:
+		virtual ~CPatchmanTester() {}
+	
 		// this is a singleton so it has a getInstance() method to get to the singleton instance
 		static CPatchmanTester& getInstance();
 

code/ryzom/tools/stats_scan/char_filter_factory.h

@@ -39,6 +39,7 @@ class CStatsScanCharacter;
 class ICharFilter: public NLMISC::CRefCount
 {
 public:
+	virtual ~ICharFilter() {}
 	virtual std::string toString() const=0;
 	virtual bool evaluate(const CStatsScanCharacter* c)=0;
 };

code/ryzom/tools/stats_scan/char_info_extractor_factory.h

@@ -41,6 +41,7 @@ class CStatsScanCharacter;
 class ICharInfoExtractor: public NLMISC::CRefCount
 {
 public:
+	virtual ~ICharInfoExtractor() {}
 	virtual std::string toString() const=0;
 	virtual void execute(CCharacterScanJob* job,const CStatsScanCharacter* c)=0;
 };

code/ryzom/tools/stats_scan/char_scan_script.h

@@ -97,6 +97,7 @@ private:
 class ICharScanScriptCommand: public NLMISC::CRefCount
 {
 public:
+	virtual ~ICharScanScriptCommand() {}
 	virtual const char* getName()=0;
 	virtual const char* getSyntax()=0;
 	virtual const char* getDescription()=0;