1) some cleanup in wxHtmlWindow, moved private structures out of headers

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

diff --git a/contrib/src/xml/xmlres.cpp b/contrib/src/xml/xmlres.cpp
index 7c204def23d6c2e65864daf816204337041df2c0..19c6eaf4cf46d0d2064784ac00b825df5dfe5e94 100644 (file)
--- a/contrib/src/xml/xmlres.cpp
+++ b/contrib/src/xml/xmlres.cpp
@@ -21,6 +21,7 @@
 
 #include "wx/dialog.h"
 #include "wx/panel.h"
 
 #include "wx/dialog.h"
 #include "wx/panel.h"

+#include "wx/frame.h"

 #include "wx/wfstream.h"
 #include "wx/filesys.h"
 #include "wx/log.h"
 #include "wx/wfstream.h"
 #include "wx/filesys.h"
 #include "wx/log.h"


@@ -254,7 +255,7 @@ void wxXmlResource::UpdateResources()
 {
     bool modif;
 #   if wxUSE_FILESYSTEM
 {
     bool modif;
 #   if wxUSE_FILESYSTEM

-    wxFSFile *file;
+    wxFSFile *file = NULL;

     wxFileSystem fsys;
 #   endif
 
     wxFileSystem fsys;
 #   endif
 


@@ -269,7 +270,7 @@ void wxXmlResource::UpdateResources()
             modif = file && file->GetModificationTime() > m_Data[i].Time;
             if (!file)
                 wxLogError(_("Cannot open file '%s'."), m_Data[i].File.c_str());
             modif = file && file->GetModificationTime() > m_Data[i].Time;
             if (!file)
                 wxLogError(_("Cannot open file '%s'."), m_Data[i].File.c_str());

-            delete file;
+            wxDELETE(file);

 #           else
             modif = wxDateTime(wxFileModificationTime(m_Data[i].File)) > m_Data[i].Time;
 #           endif
 #           else
             modif = wxDateTime(wxFileModificationTime(m_Data[i].File)) > m_Data[i].Time;
 #           endif


@@ -277,11 +278,12 @@ void wxXmlResource::UpdateResources()
         
         if (modif)
         {
         
         if (modif)
         {

-            wxInputStream *stream;
+                       wxInputStream *stream = NULL;

 
 #           if wxUSE_FILESYSTEM
             file = fsys.OpenFile(m_Data[i].File);
 
 #           if wxUSE_FILESYSTEM
             file = fsys.OpenFile(m_Data[i].File);

-            stream = file->GetStream();
+                       if (file)
+                               stream = file->GetStream();

 #           else
             stream = new wxFileInputStream(m_Data[i].File);
 #           endif
 #           else
             stream = new wxFileInputStream(m_Data[i].File);
 #           endif


@@ -294,22 +296,23 @@ void wxXmlResource::UpdateResources()
             if (!stream || !m_Data[i].Doc->Load(*stream))
             {
                 wxLogError(_("Cannot load resources from file '%s'."), m_Data[i].File.c_str());
             if (!stream || !m_Data[i].Doc->Load(*stream))
             {
                 wxLogError(_("Cannot load resources from file '%s'."), m_Data[i].File.c_str());

-                delete m_Data[i].Doc;
-                m_Data[i].Doc = NULL;
+                wxDELETE(m_Data[i].Doc);

             }
             else if (m_Data[i].Doc->GetRoot()->GetName() != wxT("resource")) 
             {
                 wxLogError(_("Invalid XML resource '%s': doesn't have root node 'resource'."), m_Data[i].File.c_str());
             }
             else if (m_Data[i].Doc->GetRoot()->GetName() != wxT("resource")) 
             {
                 wxLogError(_("Invalid XML resource '%s': doesn't have root node 'resource'."), m_Data[i].File.c_str());

-                delete m_Data[i].Doc;
-                m_Data[i].Doc = NULL;
+                wxDELETE(m_Data[i].Doc);

             }
             else
             }
             else

+                       {

                 ProcessPlatformProperty(m_Data[i].Doc->GetRoot());
                 ProcessPlatformProperty(m_Data[i].Doc->GetRoot());

+                               m_Data[i].Time = file->GetModificationTime();
+                       }

 
 #           if wxUSE_FILESYSTEM
 
 #           if wxUSE_FILESYSTEM

-            delete file;
+                               wxDELETE(file);

 #           else
 #           else

-            delete stream;
+                               wxDELETE(stream);

 #           endif
         }
     }
 #           endif
         }
     }


@@ -560,7 +563,7 @@ wxColour wxXmlResourceHandler::GetColour(const wxString& param)
     wxString v = GetParamValue(param);
     unsigned long tmp = 0; 
     
     wxString v = GetParamValue(param);
     unsigned long tmp = 0; 
     

-    if (v.Length() != 7 || v[0] != wxT('#') ||
+    if (v.Length() != 7 || v[0u] != wxT('#') ||

         wxSscanf(v.c_str(), wxT("#%lX"), &tmp) != 1)
     {
         wxLogError(_("XML resource: Incorrect colour specification '%s' for property '%s'."),
         wxSscanf(v.c_str(), wxT("#%lX"), &tmp) != 1)
     {
         wxLogError(_("XML resource: Incorrect colour specification '%s' for property '%s'."),