]> git.saurik.com Git - wxWidgets.git/blobdiff - src/html/htmlfilter.cpp
fix leak introduced by r53753
[wxWidgets.git] / src / html / htmlfilter.cpp
index 792240be7e4f6bf98ba3429de6622e1bc65de303..57508b1069a7386b9ec61df99769aeed5556c288 100644 (file)
@@ -8,12 +8,11 @@
 
 
 #ifdef __GNUG__
 
 
 #ifdef __GNUG__
-#pragma implementation
+#pragma implementation "htmlfilter.h"
 #endif
 
 #endif
 
-#include <wx/wxprec.h>
+#include "wx/wxprec.h"
 
 
-#include "wx/defs.h"
 #if wxUSE_HTML
 
 #ifdef __BORDLANDC__
 #if wxUSE_HTML
 
 #ifdef __BORDLANDC__
 #endif
 
 #ifndef WXPRECOMP
 #endif
 
 #ifndef WXPRECOMP
-#include <wx/wx.h>
 #endif
 
 #endif
 
-#include <wx/html/htmlfilter.h>
-#include <wx/html/htmlwin.h>
+#include "wx/html/htmlfilter.h"
+#include "wx/html/htmlwin.h"
 
 
 /*
 
 
 /*
@@ -43,29 +41,29 @@ IMPLEMENT_ABSTRACT_CLASS(wxHtmlFilter, wxObject)
 
 IMPLEMENT_DYNAMIC_CLASS(wxHtmlFilterPlainText, wxHtmlFilter)
 
 
 IMPLEMENT_DYNAMIC_CLASS(wxHtmlFilterPlainText, wxHtmlFilter)
 
-bool wxHtmlFilterPlainText::CanRead(const wxFSFile& file)
+bool wxHtmlFilterPlainText::CanRead(const wxFSFile& WXUNUSED(file)) const
 {
     return TRUE;
 }
 
 
 
 {
     return TRUE;
 }
 
 
 
-wxString wxHtmlFilterPlainText::ReadFile(const wxFSFile& file)
+wxString wxHtmlFilterPlainText::ReadFile(const wxFSFile& file) const
 {
     wxInputStream *s = file.GetStream();
     char *src;
     wxString doc, doc2;
 
     if (s == NULL) return wxEmptyString;
 {
     wxInputStream *s = file.GetStream();
     char *src;
     wxString doc, doc2;
 
     if (s == NULL) return wxEmptyString;
-    src = (char*) malloc(s -> StreamSize());
-    src[s -> StreamSize()] = 0;
-    s -> Read(src, s -> StreamSize());
+    src = new char[s -> GetSize()+1];
+    src[s -> GetSize()] = 0;
+    s -> Read(src, s -> GetSize());
     doc = src;
     doc = src;
-    free(src);
+    delete [] src;
 
 
-    doc.Replace("<", "&lt;", TRUE);
-    doc.Replace(">", "&gt;", TRUE);
-    doc2 = "<HTML><BODY><PRE>\n" + doc + "\n</PRE></BODY></HTML>";
+    doc.Replace(_T("<"), _T("&lt;"), TRUE);
+    doc.Replace(_T(">"), _T("&gt;"), TRUE);
+    doc2 = _T("<HTML><BODY><PRE>\n") + doc + _T("\n</PRE></BODY></HTML>");
     return doc2;
 }
 
     return doc2;
 }
 
@@ -83,22 +81,22 @@ class wxHtmlFilterImage : public wxHtmlFilter
     DECLARE_DYNAMIC_CLASS(wxHtmlFilterImage)
 
     public:
     DECLARE_DYNAMIC_CLASS(wxHtmlFilterImage)
 
     public:
-        virtual bool CanRead(const wxFSFile& file);
-        virtual wxString ReadFile(const wxFSFile& file);
+        virtual bool CanRead(const wxFSFile& file) const;
+        virtual wxString ReadFile(const wxFSFile& file) const;
 };
 
 IMPLEMENT_DYNAMIC_CLASS(wxHtmlFilterImage, wxHtmlFilter)
 
 
 
 };
 
 IMPLEMENT_DYNAMIC_CLASS(wxHtmlFilterImage, wxHtmlFilter)
 
 
 
-bool wxHtmlFilterImage::CanRead(const wxFSFile& file)
+bool wxHtmlFilterImage::CanRead(const wxFSFile& file) const
 {
     return (file.GetMimeType().Left(6) == "image/");
 }
 
 
 
 {
     return (file.GetMimeType().Left(6) == "image/");
 }
 
 
 
-wxString wxHtmlFilterImage::ReadFile(const wxFSFile& file)
+wxString wxHtmlFilterImage::ReadFile(const wxFSFile& file) const
 {
     return ("<HTML><BODY><IMG SRC=\"" + file.GetLocation() + "\"></BODY></HTML>");
 }
 {
     return ("<HTML><BODY><IMG SRC=\"" + file.GetLocation() + "\"></BODY></HTML>");
 }
@@ -116,32 +114,36 @@ class wxHtmlFilterHTML : public wxHtmlFilter
     DECLARE_DYNAMIC_CLASS(wxHtmlFilterHTML)
 
     public:
     DECLARE_DYNAMIC_CLASS(wxHtmlFilterHTML)
 
     public:
-        virtual bool CanRead(const wxFSFile& file);
-        virtual wxString ReadFile(const wxFSFile& file);
+        virtual bool CanRead(const wxFSFile& file) const;
+        virtual wxString ReadFile(const wxFSFile& file) const;
 };
 
 
 IMPLEMENT_DYNAMIC_CLASS(wxHtmlFilterHTML, wxHtmlFilter)
 
 };
 
 
 IMPLEMENT_DYNAMIC_CLASS(wxHtmlFilterHTML, wxHtmlFilter)
 
-bool wxHtmlFilterHTML::CanRead(const wxFSFile& file)
+bool wxHtmlFilterHTML::CanRead(const wxFSFile& file) const
 {
 {
-    return (file.GetMimeType() == "text/html");
+//    return (file.GetMimeType() == "text/html");
+// This is true in most case but some page can return:
+// "text/html; char-encoding=...."
+// So we use Find instead
+  return (file.GetMimeType().Find(_T("text/html")) == 0);
 }
 
 
 
 }
 
 
 
-wxString wxHtmlFilterHTML::ReadFile(const wxFSFile& file)
+wxString wxHtmlFilterHTML::ReadFile(const wxFSFile& file) const
 {
     wxInputStream *s = file.GetStream();
     char *src;
     wxString doc;
 
     if (s == NULL) return wxEmptyString;
 {
     wxInputStream *s = file.GetStream();
     char *src;
     wxString doc;
 
     if (s == NULL) return wxEmptyString;
-    src = (char*) malloc(s -> StreamSize() + 1);
-    src[s -> StreamSize()] = 0;
-    s -> Read(src, s -> StreamSize());
+    src = new char[s -> GetSize() + 1];
+    src[s -> GetSize()] = 0;
+    s -> Read(src, s -> GetSize());
     doc = src;
     doc = src;
-    free(src);
+    delete[] src;
 
     return doc;
 }
 
     return doc;
 }
@@ -167,4 +169,4 @@ class wxHtmlFilterModule : public wxModule
 
 IMPLEMENT_DYNAMIC_CLASS(wxHtmlFilterModule, wxModule)
 
 
 IMPLEMENT_DYNAMIC_CLASS(wxHtmlFilterModule, wxModule)
 
-#endif
\ No newline at end of file
+#endif