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

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;
 }

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:

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;
 }

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:

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;
 }

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;
   

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;
 }
 
 

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);