From 44ba4cfdb6471506db37634b032cf4fa3784f4f9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Peter=20K=C3=BCmmel?= Date: Wed, 18 Jul 2012 12:17:39 +0200 Subject: [PATCH] Ninja: remove warnings --- Source/cmMakefileTargetGenerator.cxx | 19 +++++++++---------- Source/cmMakefileTargetGenerator.h | 16 ++++++++-------- Source/cmNinjaTargetGenerator.cxx | 15 +++++---------- Source/cmNinjaTargetGenerator.h | 13 +++++++------ Source/cmOSXBundleGenerator.h | 5 +++-- 5 files changed, 32 insertions(+), 36 deletions(-) diff --git a/Source/cmMakefileTargetGenerator.cxx b/Source/cmMakefileTargetGenerator.cxx index bc016dc34..0de182ede 100644 --- a/Source/cmMakefileTargetGenerator.cxx +++ b/Source/cmMakefileTargetGenerator.cxx @@ -29,7 +29,7 @@ cmMakefileTargetGenerator::cmMakefileTargetGenerator(cmTarget* target) : OSXBundleGenerator(0) - , MacOSXContentGenerator(this) + , MacOSXContentGenerator(0) { this->BuildFileStream = 0; this->InfoFileStream = 0; @@ -52,6 +52,12 @@ cmMakefileTargetGenerator::cmMakefileTargetGenerator(cmTarget* target) { this->NoRuleMessages = cmSystemTools::IsOff(ruleStatus); } + MacOSXContentGenerator = new MacOSXContentGeneratorType(this); +} + +cmMakefileTargetGenerator::~cmMakefileTargetGenerator() +{ + delete MacOSXContentGenerator; } cmMakefileTargetGenerator * @@ -157,10 +163,10 @@ void cmMakefileTargetGenerator::WriteTargetBuildRules() } this->OSXBundleGenerator->GenerateMacOSXContentStatements( this->GeneratorTarget->HeaderSources, - &this->MacOSXContentGenerator); + this->MacOSXContentGenerator); this->OSXBundleGenerator->GenerateMacOSXContentStatements( this->GeneratorTarget->ExtraSources, - &this->MacOSXContentGenerator); + this->MacOSXContentGenerator); for(std::vector::const_iterator si = this->GeneratorTarget->ExternalObjects.begin(); si != this->GeneratorTarget->ExternalObjects.end(); ++si) @@ -348,13 +354,6 @@ void cmMakefileTargetGenerator::WriteTargetLanguageFlags() } } -//---------------------------------------------------------------------------- -cmMakefileTargetGenerator::MacOSXContentGeneratorType:: -MacOSXContentGeneratorType(cmMakefileTargetGenerator* generator) - : cmOSXBundleGenerator::MacOSXContentGeneratorType() - , Generator(generator) -{ -} //---------------------------------------------------------------------------- void diff --git a/Source/cmMakefileTargetGenerator.h b/Source/cmMakefileTargetGenerator.h index d5eb634e7..3ea524bfa 100644 --- a/Source/cmMakefileTargetGenerator.h +++ b/Source/cmMakefileTargetGenerator.h @@ -35,7 +35,7 @@ class cmMakefileTargetGenerator public: // constructor to set the ivars cmMakefileTargetGenerator(cmTarget* target); - virtual ~cmMakefileTargetGenerator() {}; + virtual ~cmMakefileTargetGenerator(); // construct using this factory call static cmMakefileTargetGenerator *New(cmTarget *tgt); @@ -74,17 +74,17 @@ protected: void WriteTargetDependRules(); // write rules for Mac OS X Application Bundle content. - class MacOSXContentGeneratorType - : public cmOSXBundleGenerator::MacOSXContentGeneratorType + struct MacOSXContentGeneratorType : + cmOSXBundleGenerator::MacOSXContentGeneratorType { - public: - MacOSXContentGeneratorType(cmMakefileTargetGenerator* Generator); - virtual void operator()(cmSourceFile& source, const char* pkgloc); + MacOSXContentGeneratorType(cmMakefileTargetGenerator* gen) : + Generator(gen) {} + + void operator()(cmSourceFile& source, const char* pkgloc); private: cmMakefileTargetGenerator* Generator; }; - friend class MacOSXContentGeneratorType; // write the rules for an object void WriteObjectRuleFiles(cmSourceFile& source); @@ -234,7 +234,7 @@ protected: std::string MacContentDirectory; std::set MacContentFolders; cmOSXBundleGenerator* OSXBundleGenerator; - MacOSXContentGeneratorType MacOSXContentGenerator; + MacOSXContentGeneratorType* MacOSXContentGenerator; typedef std::map ByLanguageMap; std::string GetFlags(const std::string &l); diff --git a/Source/cmNinjaTargetGenerator.cxx b/Source/cmNinjaTargetGenerator.cxx index ac2b4688b..967507ed7 100644 --- a/Source/cmNinjaTargetGenerator.cxx +++ b/Source/cmNinjaTargetGenerator.cxx @@ -57,7 +57,7 @@ cmNinjaTargetGenerator::New(cmTarget* target) cmNinjaTargetGenerator::cmNinjaTargetGenerator(cmTarget* target) : - MacOSXContentGenerator(this), + MacOSXContentGenerator(0), OSXBundleGenerator(0), MacContentFolders(), Target(target), @@ -68,10 +68,12 @@ cmNinjaTargetGenerator::cmNinjaTargetGenerator(cmTarget* target) { this->GeneratorTarget = this->GetGlobalGenerator()->GetGeneratorTarget(target); + MacOSXContentGenerator = new MacOSXContentGeneratorType(this); } cmNinjaTargetGenerator::~cmNinjaTargetGenerator() { + delete MacOSXContentGenerator; } cmGeneratedFileStream& cmNinjaTargetGenerator::GetBuildFileStream() const @@ -434,10 +436,10 @@ cmNinjaTargetGenerator } this->OSXBundleGenerator->GenerateMacOSXContentStatements( this->GeneratorTarget->HeaderSources, - &this->MacOSXContentGenerator); + this->MacOSXContentGenerator); this->OSXBundleGenerator->GenerateMacOSXContentStatements( this->GeneratorTarget->ExtraSources, - &this->MacOSXContentGenerator); + this->MacOSXContentGenerator); for(std::vector::const_iterator si = this->GeneratorTarget->ExternalObjects.begin(); si != this->GeneratorTarget->ExternalObjects.end(); ++si) @@ -645,13 +647,6 @@ cmNinjaTargetGenerator EnsureDirectoryExists(cmSystemTools::GetParentDirectory(path.c_str())); } -//---------------------------------------------------------------------------- -cmNinjaTargetGenerator::MacOSXContentGeneratorType:: -MacOSXContentGeneratorType(cmNinjaTargetGenerator* generator) - : cmOSXBundleGenerator::MacOSXContentGeneratorType() - , Generator(generator) -{ -} //---------------------------------------------------------------------------- void diff --git a/Source/cmNinjaTargetGenerator.h b/Source/cmNinjaTargetGenerator.h index a2ca4bd8c..fb4583746 100644 --- a/Source/cmNinjaTargetGenerator.h +++ b/Source/cmNinjaTargetGenerator.h @@ -116,12 +116,13 @@ protected: void EnsureParentDirectoryExists(const std::string& path); // write rules for Mac OS X Application Bundle content. - class MacOSXContentGeneratorType - : public cmOSXBundleGenerator::MacOSXContentGeneratorType + struct MacOSXContentGeneratorType : + cmOSXBundleGenerator::MacOSXContentGeneratorType { - public: - MacOSXContentGeneratorType(cmNinjaTargetGenerator* Generator); - virtual void operator()(cmSourceFile& source, const char* pkgloc); + MacOSXContentGeneratorType(cmNinjaTargetGenerator* g) : + Generator(g) {} + + void operator()(cmSourceFile& source, const char* pkgloc); private: cmNinjaTargetGenerator* Generator; @@ -129,7 +130,7 @@ protected: protected: - MacOSXContentGeneratorType MacOSXContentGenerator; + MacOSXContentGeneratorType* MacOSXContentGenerator; // Properly initialized by sub-classes. cmOSXBundleGenerator* OSXBundleGenerator; std::set MacContentFolders; diff --git a/Source/cmOSXBundleGenerator.h b/Source/cmOSXBundleGenerator.h index 2ed9e91af..01e3cbeb4 100644 --- a/Source/cmOSXBundleGenerator.h +++ b/Source/cmOSXBundleGenerator.h @@ -35,9 +35,9 @@ public: void CreateFramework(std::string const& targetName); void CreateCFBundle(std::string& targetName, std::string& outpath); - class MacOSXContentGeneratorType + struct MacOSXContentGeneratorType { - public: + virtual ~MacOSXContentGeneratorType() {} virtual void operator()(cmSourceFile& source, const char* pkgloc) = 0; }; @@ -67,4 +67,5 @@ private: std::set* MacContentFolders; }; + #endif