summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBen Boeckel <ben.boeckel@kitware.com>2010-09-02 15:29:05 (GMT)
committerBen Boeckel <ben.boeckel@kitware.com>2010-09-02 15:29:05 (GMT)
commitd784e6af4d292e5ffee4b43b7ee740d1fa8e2e29 (patch)
treeab20997a3378ac991c9800784423e2d56b7c8a9b
parent9efc05722e9598b8e34062e8f39457b728b7286b (diff)
downloadCMake-d784e6af4d292e5ffee4b43b7ee740d1fa8e2e29.zip
CMake-d784e6af4d292e5ffee4b43b7ee740d1fa8e2e29.tar.gz
CMake-d784e6af4d292e5ffee4b43b7ee740d1fa8e2e29.tar.bz2
Run the unused variables check on the final pass
-rw-r--r--Source/cmMakefile.cxx2
-rw-r--r--Source/cmake.cxx34
-rw-r--r--Source/cmake.h2
3 files changed, 23 insertions, 15 deletions
diff --git a/Source/cmMakefile.cxx b/Source/cmMakefile.cxx
index 3a23590..288bc53 100644
--- a/Source/cmMakefile.cxx
+++ b/Source/cmMakefile.cxx
@@ -828,6 +828,8 @@ void cmMakefile::ConfigureFinalPass()
{
l->second.FinishConfigure();
}
+
+ this->GetCMakeInstance()->RunCheckForUnusedVariables();
}
//----------------------------------------------------------------------------
diff --git a/Source/cmake.cxx b/Source/cmake.cxx
index cb70f1f..b3261c5 100644
--- a/Source/cmake.cxx
+++ b/Source/cmake.cxx
@@ -201,21 +201,6 @@ cmake::cmake()
cmake::~cmake()
{
-#ifdef CMAKE_BUILD_WITH_CMAKE
- if(this->WarnUnusedCli)
- {
- std::map<std::string, bool>::const_iterator it;
- for(it = this->UsedCliVariables.begin(); it != this->UsedCliVariables.end(); ++it)
- {
- if(!it->second)
- {
- std::string message = "warning: The variable, \"" + it->first + "\", given "
- "on the command line, was not used within the build.";
- cmSystemTools::Message(message.c_str());
- }
- }
- }
-#endif
delete this->CacheManager;
delete this->Policies;
if (this->GlobalGenerator)
@@ -4500,3 +4485,22 @@ int cmake::Build(const std::string& dir,
config.c_str(), clean, false, 0, true,
0, nativeOptions);
}
+
+void cmake::RunCheckForUnusedVariables() const
+{
+#ifdef CMAKE_BUILD_WITH_CMAKE
+ if(this->WarnUnusedCli)
+ {
+ std::map<std::string, bool>::const_iterator it;
+ for(it = this->UsedCliVariables.begin(); it != this->UsedCliVariables.end(); ++it)
+ {
+ if(!it->second)
+ {
+ std::string message = "warning: The variable, \"" + it->first + "\", given "
+ "on the command line, was not used within the build.";
+ cmSystemTools::Message(message.c_str());
+ }
+ }
+ }
+#endif
+}
diff --git a/Source/cmake.h b/Source/cmake.h
index 403809f..4277bdd 100644
--- a/Source/cmake.h
+++ b/Source/cmake.h
@@ -362,6 +362,8 @@ class cmake
const std::string& config,
const std::vector<std::string>& nativeOptions,
bool clean);
+
+ void RunCheckForUnusedVariables() const;
protected:
void InitializeProperties();
int HandleDeleteCacheVariables(const char* var);