ENH: Changed WriteConfiguration back to const because it doesn't need to report errors anymore.

8 files changed, 10 insertions, 14 deletions

diff --git a/Source/cmCableInstantiateClassCommand.cxx b/Source/cmCableInstantiateClassCommand.cxx
index 179ea8e..f6db806 100644
--- a/Source/cmCableInstantiateClassCommand.cxx
+++ b/Source/cmCableInstantiateClassCommand.cxx
@@ -48,7 +48,7 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  * Write the CABLE configuration code to define this InstantiationSet.
  * This includes the "class" keyword to do class template instantiations.
  */
-bool cmCableInstantiateClassCommand::WriteConfiguration()
+void cmCableInstantiateClassCommand::WriteConfiguration() const
 {
   std::ostream& os = m_CableData->GetOutputStream();
   cmCableData::Indentation indent = m_CableData->GetIndentation();
@@ -71,6 +71,4 @@ bool cmCableInstantiateClassCommand::WriteConfiguration()
     os << "</Element>" << std::endl;
     }
   os << indent << "</InstantiationSet>" << std::endl;
-  
-  return true;
 }
diff --git a/Source/cmCableInstantiateClassCommand.h b/Source/cmCableInstantiateClassCommand.h
index 173acb1..1d54e6a 100644
--- a/Source/cmCableInstantiateClassCommand.h
+++ b/Source/cmCableInstantiateClassCommand.h
@@ -88,7 +88,7 @@ public:
       "template classes (not functions, operators, etc).";
     }
 
-  virtual bool WriteConfiguration();
+  virtual void WriteConfiguration() const;
 
   cmTypeMacro(cmCableInstantiateClassCommand, cmCableInstantiateCommand);
 protected:
diff --git a/Source/cmCableInstantiateCommand.cxx b/Source/cmCableInstantiateCommand.cxx
index a0dd898..ddf7903 100644
--- a/Source/cmCableInstantiateCommand.cxx
+++ b/Source/cmCableInstantiateCommand.cxx
@@ -47,7 +47,7 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 /**
  * Write the CABLE configuration code to define this InstantiationSet.
  */
-bool cmCableInstantiateCommand::WriteConfiguration()
+void cmCableInstantiateCommand::WriteConfiguration() const
 {
   std::ostream& os = m_CableData->GetOutputStream();
   cmCableData::Indentation indent = m_CableData->GetIndentation();
@@ -70,6 +70,4 @@ bool cmCableInstantiateCommand::WriteConfiguration()
     os << "</Element>" << std::endl;
     }
   os << indent << "</InstantiationSet>" << std::endl;
-  
-  return true;
 }
diff --git a/Source/cmCableInstantiateCommand.h b/Source/cmCableInstantiateCommand.h
index b319b5e..eb173eb 100644
--- a/Source/cmCableInstantiateCommand.h
+++ b/Source/cmCableInstantiateCommand.h
@@ -87,7 +87,7 @@ public:
       "template non-classes (functions, operators, etc).";
     }
 
-  virtual bool WriteConfiguration();
+  virtual void WriteConfiguration() const;
 
   cmTypeMacro(cmCableInstantiateCommand, cmCablePackageCommand);
 protected:
diff --git a/Source/cmCablePackageEntryCommand.cxx b/Source/cmCablePackageEntryCommand.cxx
index 771966a..c8873bb 100644
--- a/Source/cmCablePackageEntryCommand.cxx
+++ b/Source/cmCablePackageEntryCommand.cxx
@@ -61,5 +61,7 @@ bool cmCablePackageEntryCommand::Invoke(std::vector<std::string>& args)
     }  
   
   // Write this command's configuration.
-  return this->WriteConfiguration();
+  this->WriteConfiguration();
+  
+  return true;
 }
diff --git a/Source/cmCablePackageEntryCommand.h b/Source/cmCablePackageEntryCommand.h
index 38bc9d5..3759347 100644
--- a/Source/cmCablePackageEntryCommand.h
+++ b/Source/cmCablePackageEntryCommand.h
@@ -67,7 +67,7 @@ public:
 
   cmTypeMacro(cmCablePackageEntryCommand, cmCableCommand);
 
-  virtual bool WriteConfiguration() =0;
+  virtual void WriteConfiguration() const =0;
 protected:
   typedef std::vector<std::string>  Entries;
   
diff --git a/Source/cmCableSourceFilesCommand.cxx b/Source/cmCableSourceFilesCommand.cxx
index 4daff3c..c4024f8 100644
--- a/Source/cmCableSourceFilesCommand.cxx
+++ b/Source/cmCableSourceFilesCommand.cxx
@@ -70,7 +70,7 @@ void cmCableSourceFilesCommand::FinalPass()
  * Write the CABLE configuration code to indicate header dependencies for
  * a package.
  */
-bool cmCableSourceFilesCommand::WriteConfiguration()
+void cmCableSourceFilesCommand::WriteConfiguration() const
 {
   std::ostream& os = m_CableData->GetOutputStream();
   cmCableData::Indentation indent = m_CableData->GetIndentation();
@@ -102,8 +102,6 @@ bool cmCableSourceFilesCommand::WriteConfiguration()
       }
     }
   os << indent << "</Headers>" << std::endl;
-  
-  return true;
 }
 
 
diff --git a/Source/cmCableSourceFilesCommand.h b/Source/cmCableSourceFilesCommand.h
index 5c7cdce..f5d77de 100644
--- a/Source/cmCableSourceFilesCommand.h
+++ b/Source/cmCableSourceFilesCommand.h
@@ -92,7 +92,7 @@ public:
       "Generates a Package's Headers block in the CABLE configuration.";
     }
 
-  virtual bool WriteConfiguration();
+  virtual void WriteConfiguration() const;
   bool SourceFileExists(const std::string&) const;
   
   cmTypeMacro(cmCableSourceFilesCommand, cmCableCommand);