From bac5b8d08474d36b12014dfa774f0b88374d0299 Mon Sep 17 00:00:00 2001 From: kervala Date: Sat, 7 Apr 2012 23:47:55 +0200 Subject: [PATCH] Changed: #825 Remove all warnings when compiling Ryzom --- code/ryzom/common/src/game_share/singleton_registry.h | 1 + .../src/entities_game_service/pvp_manager/pvp_interface.h | 3 +-- .../server/src/general_utilities_service/gus_client_manager.h | 1 + .../server/src/general_utilities_service/rs_remote_saves.h | 2 ++ code/ryzom/server/src/general_utilities_service/saves_unit.h | 3 +++ code/ryzom/server/src/patchman_service/patchman_tester.h | 2 ++ code/ryzom/tools/stats_scan/char_filter_factory.h | 1 + code/ryzom/tools/stats_scan/char_info_extractor_factory.h | 1 + code/ryzom/tools/stats_scan/char_scan_script.h | 1 + 9 files changed, 13 insertions(+), 2 deletions(-) diff --git a/code/ryzom/common/src/game_share/singleton_registry.h b/code/ryzom/common/src/game_share/singleton_registry.h index 516417c32..7a9cc0253 100644 --- a/code/ryzom/common/src/game_share/singleton_registry.h +++ b/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 diff --git a/code/ryzom/server/src/entities_game_service/pvp_manager/pvp_interface.h b/code/ryzom/server/src/entities_game_service/pvp_manager/pvp_interface.h index 8524320df..dd5c92b68 100644 --- a/code/ryzom/server/src/entities_game_service/pvp_manager/pvp_interface.h +++ b/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 diff --git a/code/ryzom/server/src/general_utilities_service/gus_client_manager.h b/code/ryzom/server/src/general_utilities_service/gus_client_manager.h index c6f8b3f07..6714e4185 100644 --- a/code/ryzom/server/src/general_utilities_service/gus_client_manager.h +++ b/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; }; diff --git a/code/ryzom/server/src/general_utilities_service/rs_remote_saves.h b/code/ryzom/server/src/general_utilities_service/rs_remote_saves.h index 5ab25b8ac..1d91279bf 100644 --- a/code/ryzom/server/src/general_utilities_service/rs_remote_saves.h +++ b/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 diff --git a/code/ryzom/server/src/general_utilities_service/saves_unit.h b/code/ryzom/server/src/general_utilities_service/saves_unit.h index 3ea79287a..9685aa7bb 100644 --- a/code/ryzom/server/src/general_utilities_service/saves_unit.h +++ b/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 diff --git a/code/ryzom/server/src/patchman_service/patchman_tester.h b/code/ryzom/server/src/patchman_service/patchman_tester.h index ccab3d800..16a620c1c 100644 --- a/code/ryzom/server/src/patchman_service/patchman_tester.h +++ b/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(); diff --git a/code/ryzom/tools/stats_scan/char_filter_factory.h b/code/ryzom/tools/stats_scan/char_filter_factory.h index 642c0dfde..35ee52a07 100644 --- a/code/ryzom/tools/stats_scan/char_filter_factory.h +++ b/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; }; diff --git a/code/ryzom/tools/stats_scan/char_info_extractor_factory.h b/code/ryzom/tools/stats_scan/char_info_extractor_factory.h index 920c4344b..35e3e7b68 100644 --- a/code/ryzom/tools/stats_scan/char_info_extractor_factory.h +++ b/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; }; diff --git a/code/ryzom/tools/stats_scan/char_scan_script.h b/code/ryzom/tools/stats_scan/char_scan_script.h index 5f4a4d107..ac3a6780b 100644 --- a/code/ryzom/tools/stats_scan/char_scan_script.h +++ b/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;