wxRTC doc updates

[wxWidgets.git] / src / xml / xml.cpp

diff --git a/src/xml/xml.cpp b/src/xml/xml.cpp
index 0e82fffd01181cfef98e95292364764c31d6efe5..252331cae0f18b6cbfe49534250b8e52c7524d38 100644 (file)
--- a/src/xml/xml.cpp
+++ b/src/xml/xml.cpp
@@ -451,7 +451,7 @@ void wxXmlDocument::DoCopy(const wxXmlDocument& doc)
 bool wxXmlDocument::Load(const wxString& filename, const wxString& encoding, int flags)
 {
     wxFileInputStream stream(filename);
 bool wxXmlDocument::Load(const wxString& filename, const wxString& encoding, int flags)
 {
     wxFileInputStream stream(filename);

-    if (!stream.Ok())
+    if (!stream.IsOk())

         return false;
     return Load(stream, encoding, flags);
 }
         return false;
     return Load(stream, encoding, flags);
 }


@@ -459,7 +459,7 @@ bool wxXmlDocument::Load(const wxString& filename, const wxString& encoding, int
 bool wxXmlDocument::Save(const wxString& filename, int indentstep) const
 {
     wxFileOutputStream stream(filename);
 bool wxXmlDocument::Save(const wxString& filename, int indentstep) const
 {
     wxFileOutputStream stream(filename);

-    if (!stream.Ok())
+    if (!stream.IsOk())

         return false;
     return Save(stream, indentstep);
 }
         return false;
     return Save(stream, indentstep);
 }


@@ -505,6 +505,12 @@ wxXmlNode *wxXmlDocument::DetachRoot()
 
 void wxXmlDocument::SetRoot(wxXmlNode *root)
 {
 
 void wxXmlDocument::SetRoot(wxXmlNode *root)
 {

+    if (root)
+    {
+        wxASSERT_MSG( root->GetType() == wxXML_ELEMENT_NODE,
+                      "Can only set an element type node as root" );
+    }
+

     wxXmlNode *node = m_docNode;
     if (node)
     {
     wxXmlNode *node = m_docNode;
     if (node)
     {


@@ -515,7 +521,7 @@ void wxXmlDocument::SetRoot(wxXmlNode *root)
             prev = node;
             node = node->GetNext();
         }
             prev = node;
             node = node->GetNext();
         }

-        if (node)
+        if (node && root)

         {
             root->SetNext( node->GetNext() );
             wxDELETE(node);
         {
             root->SetNext( node->GetNext() );
             wxDELETE(node);


@@ -528,8 +534,10 @@ void wxXmlDocument::SetRoot(wxXmlNode *root)
     else
     {
         m_docNode = new wxXmlNode(wxXML_DOCUMENT_NODE, wxEmptyString);
     else
     {
         m_docNode = new wxXmlNode(wxXML_DOCUMENT_NODE, wxEmptyString);

+        m_docNode->SetChildren(root);

     }
     }

-    root->SetParent(m_docNode);
+    if (root)
+        root->SetParent(m_docNode);

 }
 
 void wxXmlDocument::AppendToProlog(wxXmlNode *node)
 }
 
 void wxXmlDocument::AppendToProlog(wxXmlNode *node)