GCC 3.4 fixes.

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

diff --git a/src/xml/xml.cpp b/src/xml/xml.cpp
index ed0b3c0b0731ca1710c044755461b52c5b76d434..80c76829fceb212a123e9cbd1aeffd425d8a457b 100644 (file)
--- a/src/xml/xml.cpp
+++ b/src/xml/xml.cpp
@@ -8,7 +8,7 @@
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)

 #pragma implementation "xml.h"
 #endif
 
 #pragma implementation "xml.h"
 #endif
 


@@ -32,6 +32,10 @@
 
 #include "expat.h" // from Expat
 
 
 #include "expat.h" // from Expat
 

+// DLL options compatibility check:
+#include "wx/app.h"
+WX_CHECK_BUILD_OPTIONS("wxXML")
+

 //-----------------------------------------------------------------------------
 //  wxXmlNode
 //-----------------------------------------------------------------------------
 //-----------------------------------------------------------------------------
 //  wxXmlNode
 //-----------------------------------------------------------------------------


@@ -152,8 +156,8 @@ wxString wxXmlNode::GetPropVal(const wxString& propName, const wxString& default
     wxString tmp;
     if (GetPropVal(propName, &tmp))
         return tmp;
     wxString tmp;
     if (GetPropVal(propName, &tmp))
         return tmp;

-    else
-        return defaultVal;
+
+    return defaultVal;

 }
 
 void wxXmlNode::AddChild(wxXmlNode *child)
 }
 
 void wxXmlNode::AddChild(wxXmlNode *child)


@@ -366,7 +370,7 @@ inline static wxString CharToString(wxMBConv *conv,
         return str;
     }
     else
         return str;
     }
     else

-        return wxString(s, len);
+        return wxString(s, len != wxSTRING_MAXLEN ? len : strlen(s));

 #endif
 }
 
 #endif
 }
 


@@ -539,8 +543,10 @@ bool wxXmlDocument::Load(wxInputStream& stream, const wxString& encoding)
         done = (len < BUFSIZE);
         if (!XML_Parse(parser, buf, len, done))
         {
         done = (len < BUFSIZE);
         if (!XML_Parse(parser, buf, len, done))
         {

+            wxString error(XML_ErrorString(XML_GetErrorCode(parser)),
+                           *wxConvCurrent);

             wxLogError(_("XML parsing error: '%s' at line %d"),
             wxLogError(_("XML parsing error: '%s' at line %d"),

-                       XML_ErrorString(XML_GetErrorCode(parser)),
+                       error.c_str(),

                        XML_GetCurrentLineNumber(parser));
             ok = false;
             break;
                        XML_GetCurrentLineNumber(parser));
             ok = false;
             break;


@@ -549,10 +555,16 @@ bool wxXmlDocument::Load(wxInputStream& stream, const wxString& encoding)
 
     if (ok)
     {
 
     if (ok)
     {

-        SetVersion(ctx.version);
-        SetFileEncoding(ctx.encoding);
+        if (!ctx.version.IsEmpty())
+            SetVersion(ctx.version);
+        if (!ctx.encoding.IsEmpty())
+            SetFileEncoding(ctx.encoding);

         SetRoot(ctx.root);
     }
         SetRoot(ctx.root);
     }

+    else
+    {
+        delete ctx.root;
+    }

 
     XML_ParserFree(parser);
 #if !wxUSE_UNICODE
 
     XML_ParserFree(parser);
 #if !wxUSE_UNICODE


@@ -572,11 +584,16 @@ bool wxXmlDocument::Load(wxInputStream& stream, const wxString& encoding)
 
 // write string to output:
 inline static void OutputString(wxOutputStream& stream, const wxString& str,
 
 // write string to output:
 inline static void OutputString(wxOutputStream& stream, const wxString& str,

-                                wxMBConv *convMem, wxMBConv *convFile)
+#if wxUSE_UNICODE
+    wxMBConv * WXUNUSED(convMem),
+#else
+    wxMBConv *convMem,
+#endif
+    wxMBConv *convFile)

 {
     if (str.IsEmpty()) return;
 #if wxUSE_UNICODE
 {
     if (str.IsEmpty()) return;
 #if wxUSE_UNICODE

-    const wxWX2MBbuf buf(str.mb_str(convFile ? *convFile : wxConvUTF8));
+    const wxWX2MBbuf buf(str.mb_str(*(convFile ? convFile : &wxConvUTF8)));

     stream.Write((const char*)buf, strlen((const char*)buf));
 #else
     if ( convFile == NULL )
     stream.Write((const char*)buf, strlen((const char*)buf));
 #else
     if ( convFile == NULL )


@@ -592,7 +609,8 @@ inline static void OutputString(wxOutputStream& stream, const wxString& str,
 // Same as above, but create entities first.
 // Translates '<' to "&lt;", '>' to "&gt;" and '&' to "&amp;"
 static void OutputStringEnt(wxOutputStream& stream, const wxString& str,
 // Same as above, but create entities first.
 // Translates '<' to "&lt;", '>' to "&gt;" and '&' to "&amp;"
 static void OutputStringEnt(wxOutputStream& stream, const wxString& str,

-                            wxMBConv *convMem, wxMBConv *convFile)
+                            wxMBConv *convMem, wxMBConv *convFile,
+                            bool escapeQuotes = false)

 {
     wxString buf;
     size_t i, last, len;
 {
     wxString buf;
     size_t i, last, len;


@@ -604,7 +622,8 @@ static void OutputStringEnt(wxOutputStream& stream, const wxString& str,
     {
         c = str.GetChar(i);
         if (c == wxT('<') || c == wxT('>') ||
     {
         c = str.GetChar(i);
         if (c == wxT('<') || c == wxT('>') ||

-            (c == wxT('&') && str.Mid(i+1, 4) != wxT("amp;")))
+            (c == wxT('&') && str.Mid(i+1, 4) != wxT("amp;")) ||
+            (escapeQuotes && c == wxT('"')))

         {
             OutputString(stream, str.Mid(last, i - last), convMem, convFile);
             switch (c)
         {
             OutputString(stream, str.Mid(last, i - last), convMem, convFile);
             switch (c)


@@ -618,6 +637,9 @@ static void OutputStringEnt(wxOutputStream& stream, const wxString& str,
                 case wxT('&'):
                     OutputString(stream, wxT("&amp;"), NULL, NULL);
                     break;
                 case wxT('&'):
                     OutputString(stream, wxT("&amp;"), NULL, NULL);
                     break;

+                case wxT('"'):
+                    OutputString(stream, wxT("&quot;"), NULL, NULL);
+                    break;

                 default: break;
             }
             last = i + 1;
                 default: break;
             }
             last = i + 1;


@@ -653,10 +675,11 @@ static void OutputNode(wxOutputStream& stream, wxXmlNode *node, int indent,
             prop = node->GetProperties();
             while (prop)
             {
             prop = node->GetProperties();
             while (prop)
             {

-                OutputString(stream, wxT(" ") + prop->GetName() +
-                             wxT("=\"") + prop->GetValue() + wxT("\""),
+                OutputString(stream, wxT(" ") + prop->GetName() +  wxT("=\""),

                              NULL, NULL);
                              NULL, NULL);

-                // FIXME - what if prop contains '"'?
+                OutputStringEnt(stream, prop->GetValue(), NULL, NULL,
+                                true/*escapeQuotes*/);
+                OutputString(stream, wxT("\""), NULL, NULL);

                 prop = prop->GetNext();
             }
 
                 prop = prop->GetNext();
             }