ENH: a warning fix and some more cleanup

Source/cmAddTestCommand.h

@@ -74,8 +74,6 @@ public:
   
   cmTypeMacro(cmAddTestCommand, cmCommand);
 
-private:
-  std::string m_Test;
 };
 
 

Source/cmData.h

@@ -31,13 +31,13 @@
 class cmData
 {
 public:
-  cmData(const char* name): m_Name(name) {}
+  cmData(const char* name): Name(name) {}
   virtual ~cmData() {}
   
   const std::string& GetName() const
-    { return m_Name; }
+    { return this->Name; }
 protected:
-  std::string m_Name;
+  std::string Name;
 };
 
 #endif

Source/cmDependsC.cxx

@@ -38,7 +38,7 @@ cmDependsC::cmDependsC(std::vector<std::string> const& includes,
   IncludeRegexScan(scanRegex),
   IncludeRegexComplain(complainRegex),
   GeneratedFiles(&generatedFiles),
-  cacheFileName(cacheFileName)
+  CacheFileName(cacheFileName)
 {
   this->ReadCacheFile();
 }
@@ -212,11 +212,11 @@ bool cmDependsC::WriteDependencies(const char *src, const char *obj,
 //----------------------------------------------------------------------------
 void cmDependsC::ReadCacheFile()
 {
-  if(this->cacheFileName.size() == 0)
+  if(this->CacheFileName.size() == 0)
     {
     return;
     }
-  std::ifstream fin(this->cacheFileName.c_str());
+  std::ifstream fin(this->CacheFileName.c_str());
   if(!fin)
     {
     return;
@@ -240,7 +240,7 @@ void cmDependsC::ReadCacheFile()
       haveFileName=true;
       int newer=0;
       cmFileTimeComparison comp;
-      bool res=comp.FileTimeCompare(this->cacheFileName.c_str(), 
+      bool res=comp.FileTimeCompare(this->CacheFileName.c_str(), 
                                     line.c_str(), &newer);
       
       if ((res==true) && (newer==1)) //cache is newer than the parsed file
@@ -268,11 +268,11 @@ void cmDependsC::ReadCacheFile()
 //----------------------------------------------------------------------------
 void cmDependsC::WriteCacheFile() const
 {
-  if(this->cacheFileName.size() == 0)
+  if(this->CacheFileName.size() == 0)
     {
     return;
     }
-  std::ofstream cacheOut(this->cacheFileName.c_str());
+  std::ofstream cacheOut(this->CacheFileName.c_str());
   if(!cacheOut)
     {
     return;

Source/cmDependsC.h

@@ -93,7 +93,7 @@ protected:
 
   std::map<cmStdString, cmIncludeLines *> fileCache;
 
-  cmStdString cacheFileName;
+  cmStdString CacheFileName;
 
   void WriteCacheFile() const;
   void ReadCacheFile();

Source/cmStandardIncludes.h

@@ -240,11 +240,11 @@ private:
 class cmOStrStreamCleanup
 {
 public:
-  cmOStrStreamCleanup(std::ostrstream& ostr): m_OStrStream(ostr) {}
-  ~cmOStrStreamCleanup() { m_OStrStream.rdbuf()->freeze(0); }
+  cmOStrStreamCleanup(std::ostrstream& ostr): OStrStream(ostr) {}
+  ~cmOStrStreamCleanup() { this->OStrStream.rdbuf()->freeze(0); }
   static void IgnoreUnusedVariable(const cmOStrStreamCleanup&) {}
 protected:
-  std::ostrstream& m_OStrStream;
+  std::ostrstream& OStrStream;
 };
 
 class cmOStringStream: public std::ostrstream

Source/cmUseMangledMesaCommand.h

@@ -75,12 +75,6 @@ public:
 protected:
   void CopyAndFullPathMesaHeader(const char* source,
                                  const char* outdir);
-private:
-  std::vector<cmSourceFile> m_WrapClasses;
-  std::vector<std::string> m_WrapHeaders;
-  std::string m_LibraryName;
-  std::string m_SourceList;
-  std::vector<std::string> m_Commands;
 };
 
 

Source/cmVariableRequiresCommand.h

@@ -73,9 +73,6 @@ public:
     }
   
   cmTypeMacro(cmVariableRequiresCommand, cmCommand);
-private:
-  std::string m_ErrorMessage;
-  bool m_RequirementsMet;
 };