ENH: Changed cmCableWrapCommand to inherit from cmCableDefineSetCommand since they do almost exactly the same thing. Added a GetXmlTag virtual function to both classes to return what XML tag to generate in the set's output. cmCableDefineSetCommand generates a "Set" tag, and cmCableWrapCommand generates a "WrapperSet" tag. What is inside the tags is still generated by the cmCableDefineSetCommand superclass.

1 files changed, 1 insertions, 35 deletions

diff --git a/Source/cmCableWrapCommand.cxx b/Source/cmCableWrapCommand.cxx
index 7cea981..9364cb3 100644
--- a/Source/cmCableWrapCommand.cxx
+++ b/Source/cmCableWrapCommand.cxx
@@ -41,38 +41,4 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 #include "cmCableWrapCommand.h"
 #include "cmCacheManager.h"
 
-/**
- * Write the CABLE configuration code to define this WrapperSet.
- */
-bool cmCableWrapCommand::WriteConfiguration()
-{
-  if(m_Entries.size() < 2)
-    {
-    this->SetError("called with incorrect number of arguments");
-    return false;
-    }
-  
-  std::ostream& os = m_CableData->GetOutputStream();
-  cmCableData::Indentation indent = m_CableData->GetIndentation();
-  
-  cmRegularExpression needCdataBlock("[&<>]");
-  
-  Entries::const_iterator e = m_Entries.begin();
-  os << indent << "<WrapperSet name=\"" << e->c_str() << "\">" << std::endl;
-  for(++e;e != m_Entries.end(); ++e)
-    {
-    os << indent << "  <Element>";
-    if(needCdataBlock.find(e->c_str()))
-      {
-      os << "<![CDATA[" << e->c_str() << "]]>";
-      }
-    else
-      {
-      os << e->c_str();
-      }
-    os << "</Element>" << std::endl;
-    }
-  os << indent << "</WrapperSet>" << std::endl;
-  
-  return true;
-}
+// Don't need to implement anything here.